Use new spawner option for ros args

ros-controls · Aug 23, 2024 · 8a61ccd · 8a61ccd
1 parent 3592198
commit 8a61ccd
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 14 deletions.
diff --git a/example_11/bringup/launch/carlikebot.launch.py b/example_11/bringup/launch/carlikebot.launch.py
@@ -77,9 +77,6 @@ def generate_launch_description():
         executable="ros2_control_node",
         parameters=[robot_controllers],
         output="both",
-        remappings=[
-            ("/bicycle_steering_controller/tf_odometry", "/tf"),
-        ],
         condition=IfCondition(remap_odometry_tf),
     )
     control_node = Node(
@@ -114,7 +111,13 @@ def generate_launch_description():
     robot_bicycle_controller_spawner = Node(
         package="controller_manager",
         executable="spawner",
-        arguments=["bicycle_steering_controller", "--param-file", robot_controllers],
+        arguments=[
+            "bicycle_steering_controller",
+            "--param-file",
+            robot_controllers,
+            "--controller-ros-args",
+            "-r /bicycle_steering_controller/tf_odometry:=/tf",
+        ],
     )
 
     # Delay rviz start after `joint_state_broadcaster`

diff --git a/example_15/bringup/launch/rrbot_namespace.launch.py b/example_15/bringup/launch/rrbot_namespace.launch.py
@@ -69,13 +69,6 @@ def generate_launch_description():
         executable="ros2_control_node",
         namespace="/rrbot",
         parameters=[robot_description, robot_controllers],
-        remappings=[
-            (
-                # we use the remapping from a relative name to FQN /position_commands
-                "forward_position_controller/commands",
-                "/position_commands",
-            ),
-        ],
         output={
             "stdout": "screen",
             "stderr": "screen",
@@ -108,7 +101,14 @@ def generate_launch_description():
         package="controller_manager",
         executable="spawner",
         namespace="rrbot",
-        arguments=["forward_position_controller", "--param-file", robot_controllers],
+        arguments=[
+            "forward_position_controller",
+            "--param-file",
+            robot_controllers,
+            # we use the remapping from a relative name to FQN /position_commands
+            "--controller-ros-args",
+            "-r forward_position_controller/commands:=/position_commands",
+        ],
     )
 
     robot_position_trajectory_controller_spawner = Node(

diff --git a/example_2/bringup/launch/diffbot.launch.py b/example_2/bringup/launch/diffbot.launch.py
@@ -76,7 +76,7 @@ def generate_launch_description():
         parameters=[robot_controllers],
         output="both",
         remappings=[
-            ("/diffbot_base_controller/cmd_vel", "/cmd_vel"),
+            # ("/diffbot_base_controller/cmd_vel", "/cmd_vel"),
         ],
     )
     robot_state_pub_node = Node(
@@ -103,7 +103,13 @@ def generate_launch_description():
     robot_controller_spawner = Node(
         package="controller_manager",
         executable="spawner",
-        arguments=["diffbot_base_controller", "--param-file", robot_controllers],
+        arguments=[
+            "diffbot_base_controller",
+            "--param-file",
+            robot_controllers,
+            "--controller-ros-args",
+            "-r /diffbot_base_controller/cmd_vel:=/cmd_vel",
+        ],
     )
 
     # Delay rviz start after `joint_state_broadcaster`