Skip to content

Commit

Permalink
Fix use_respawn argument causing errors (#651) (#654)
Browse files Browse the repository at this point in the history
Signed-off-by: Aarav Gupta <[email protected]>
(cherry picked from commit cfd0f8c)

Co-authored-by: Aarav Gupta <[email protected]>
Co-authored-by: Alejandro Hernández Cordero <[email protected]>
  • Loading branch information
3 people authored Nov 22, 2024
1 parent 4b5f2ab commit 040caab
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion ros_gz_bridge/ros_gz_bridge/actions/ros_gz_bridge.py
Original file line number Diff line number Diff line change
Expand Up @@ -175,6 +175,9 @@ def execute(self, context: LaunchContext) -> Optional[List[Action]]:
if isinstance(self.__create_own_container, list):
self.__create_own_container = self.__create_own_container[0]

if isinstance(self.__use_respawn, list):
self.__use_respawn = self.__use_respawn[0]

# Standard node configuration
load_nodes = GroupAction(
condition=IfCondition(PythonExpression(['not ', self.__use_composition])),
Expand All @@ -185,7 +188,7 @@ def execute(self, context: LaunchContext) -> Optional[List[Action]]:
name=self.__bridge_name,
namespace=self.__namespace,
output='screen',
respawn=self.__use_respawn,
respawn=bool(self.__use_respawn),
respawn_delay=2.0,
parameters=[{'config_file': self.__config_file, **parsed_bridge_params}],
arguments=['--ros-args', '--log-level', self.__log_level],
Expand Down

0 comments on commit 040caab

Please sign in to comment.