Skip to content

Commit b267c92

Browse files
Migrate multi robot launch to use updated robot string syntax. (#659)
Signed-off-by: Leander Stephen D'Souza <leanderdsouza1234@gmail.com>
1 parent 7a6439b commit b267c92

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

migration/Jazzy.rst

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -289,7 +289,7 @@ In `PR #4795 <https://github.com/ros-navigation/navigation2/pull/4795>`_ the ``n
289289
Conform to ROS 2 launch syntax in Turtlebot 3 multi-robot launch file
290290
*********************************************************************
291291
292-
In `PR #4811 <https://github.com/ros-navigation/navigation2/pull/4811>`_ the ``cloned_multi_tb3_simulation_launch.py`` launch file was updated so that parsing the robots conforms to the ROS 2 launch file standards. This was achieved by refactoring the ``ParseMultiRobotPose`` class to be a custom launch substitution. This change allows users to pass the ``robots`` from another launch file through ``launch_arguments`` which was not possible with the old version.
292+
In `PR #5000 <https://github.com/ros-navigation/navigation2/pull/5000>`_ the ``cloned_multi_tb3_simulation_launch.py`` launch file was updated so that parsing the robots conforms to the ROS 2 launch file standards. This change allows users to pass the ``robots`` from another launch file through ``launch_arguments`` which was not possible with the old version.
293293
294294
Example for including ``cloned_multi_tb3_simulation_launch.py`` in another launch file:
295295
@@ -299,7 +299,7 @@ Example for including ``cloned_multi_tb3_simulation_launch.py`` in another launc
299299
PythonLaunchDescriptionSource(
300300
os.path.join(get_package_share_directory('nav2_bringup'), "launch", "cloned_multi_tb3_simulation_launch.py")
301301
),
302-
launch_arguments={"robots": "robot1={x: 0.5, y: 0.5, yaw: 1.5707}"}.items(),
302+
launch_arguments={"robots": "{name: 'robot1', pose: {x: 0.5, y: 0.5, yaw: 1.5707}}"}.items(),
303303
)
304304
305305
ComputePathThroughPoses, NavigateThroughPoses and other BT nodes now use nav_msgs/Goals instead of vector<PoseStamped>

0 commit comments

Comments
 (0)