Fix ros_gz_sim.launch.py when create_own_container is enabled. (#620)

Signed-off-by: Carlos Agüero <caguero@openrobotics.org>
gazebosim · Oct 17, 2024 · a4d00a9 · a4d00a9
1 parent 2d32f1b
commit a4d00a9
Showing 1 changed file with 14 additions and 13 deletions.
diff --git a/ros_gz_sim/launch/ros_gz_sim.launch.py b/ros_gz_sim/launch/ros_gz_sim.launch.py
@@ -83,6 +83,17 @@ def generate_launch_description():
         description='SDF world string'
     )
 
+    gz_server_description = IncludeLaunchDescription(
+        PythonLaunchDescriptionSource(
+            [PathJoinSubstitution([FindPackageShare('ros_gz_sim'),
+                                   'launch',
+                                   'gz_server.launch.py'])]),
+        launch_arguments=[('world_sdf_file', world_sdf_file),
+                          ('world_sdf_string', world_sdf_string),
+                          ('container_name', container_name),
+                          ('create_own_container', create_own_container),
+                          ('use_composition', use_composition), ])
+
     bridge_description = IncludeLaunchDescription(
         PythonLaunchDescriptionSource(
             [PathJoinSubstitution([FindPackageShare('ros_gz_bridge'),
@@ -92,20 +103,10 @@ def generate_launch_description():
                           ('config_file', config_file),
                           ('container_name', container_name),
                           ('namespace', namespace),
+                          ('create_own_container', str(False)),
                           ('use_composition', use_composition),
                           ('use_respawn', use_respawn),
-                          ('bridge_log_level', bridge_log_level)])
-
-    gz_server_description = IncludeLaunchDescription(
-        PythonLaunchDescriptionSource(
-            [PathJoinSubstitution([FindPackageShare('ros_gz_sim'),
-                                   'launch',
-                                   'gz_server.launch.py'])]),
-        launch_arguments=[('world_sdf_file', world_sdf_file),
-                          ('world_sdf_string', world_sdf_string),
-                          ('container_name', container_name),
-                          ('create_own_container', create_own_container),
-                          ('use_composition', use_composition), ])
+                          ('bridge_log_level', bridge_log_level), ])
 
     # Create the launch description and populate
     ld = LaunchDescription()
@@ -122,7 +123,7 @@ def generate_launch_description():
     ld.add_action(declare_world_sdf_file_cmd)
     ld.add_action(declare_world_sdf_string_cmd)
     # Add the actions to launch all of the bridge + gz_server nodes
-    ld.add_action(bridge_description)
     ld.add_action(gz_server_description)
+    ld.add_action(bridge_description)
 
     return ld