docs(turtlesim): fix incomplete remapping in section 6 for turtle2 teleop (backport #6585)#6683
Merged
Merged
Conversation
…leop (#6585) The remapping example in the "6 Remapping" section only redirected the cmd_vel topic, leaving the rotate_absolute action still targeting turtle1. As a result, arrow key movement worked correctly for turtle2, but the absolute orientation keys (F, G, B, V, C) silently continued to rotate turtle1 instead. Two changes are made: 1. The prose description is corrected from: "The way to change this behavior is by remapping the cmd_vel topic." to: "The way to change this behavior is by remapping the cmd_vel topic and the rotate_absolute action." 2. The shell command is updated to include the missing --remap flag: Before: ros2 run turtlesim turtle_teleop_key \ --ros-args --remap turtle1/cmd_vel:=turtle2/cmd_vel After: ros2 run turtlesim turtle_teleop_key \ --ros-args --remap turtle1/cmd_vel:=turtle2/cmd_vel \ --remap turtle1/rotate_absolute:=turtle2/rotate_absolute Testing: Verified by running both teleop nodes simultaneously — all keys (arrows and orientation) now correctly control their respective turtles. Signed-off-by: Sai Akella <64731868+A-C-Sai@users.noreply.github.com> (cherry picked from commit 1c23d4f)
fujitatomoya
approved these changes
May 12, 2026
|
HTML artifacts: https://github.com/ros2/ros2_documentation/actions/runs/25713278466/artifacts/6935809361. To view the resulting site:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
The remapping command in the "6 Remapping" section of the turtlesim tutorial
only remaps the
cmd_veltopic, which means arrow key movement works forturtle2but the absolute orientation keys (F, G, B, V, C) silently continueto control
turtle1because therotate_absoluteaction is not remapped.Two changes are made:
cmd_velandrotate_absolute--remap turtle1/rotate_absolute:=turtle2/rotate_absoluteto the shell commandDid you use Generative AI?
Claude (Sonnet 4.6) was used to assist with drafting the commit message and PR description.
Additional Information
The fix applies to the jazzy branch. The same issue may be present in other
supported distributions (humble, rolling) and may warrant separate PRs.
This is an automatic backport of pull request #6585 done by Mergify.