Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Notice] Nav2 Migrated to TwistStamped to replace Twist for cmd_vel Topics #1038

Open
SteveMacenski opened this issue Dec 3, 2024 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@SteveMacenski
Copy link

Hi,

As a key community project using Nav2, I wanted to drop you a note that I am in the process of changing the default cmd_vel topic type from Twist to TwistStamped in efforts of enabling a broader set of applications and safety constraints. This has been an on-going discussion over the last 18 months and now has the technology stack in GZ, ROS Control, and Nav2 to enable it.

To migrate, your robot and simulation should soon accept a TwistStamped message rather than a Twist message. We have a couple of provided utilities that could make this easier for you, by switching to use a Twist Publisher and Twist Subscriber [1-2]. This is what is now used in all Nav2 nodes to either publish or subscribe to a Twist or TwistStamped message.

If you would like to continue to use a Twist, you can set enable_stamped_cmd_vel to false in all nodes in your configuration that use Twist Publisher/Subscriber objects, but this will be shortly switched to being default true.

[1] https://github.com/ros-navigation/navigation2/blob/main/nav2_util/include/nav2_util/twist_publisher.hpp
[2] https://github.com/ros-navigation/navigation2/blob/main/nav2_util/include/nav2_util/twist_subscriber.hpp

Nav2 migration:

@robotpilot
Copy link
Member

Hi @SteveMacenski
I support the changing the default cmd_vel topic type from Twist to TwistStamped in the Nav2 community and will be migrating to this change in TurtleBot3.

@SteveMacenski
Copy link
Author

BTW: This is for Kilted / Rolling by default, but can be enabled in Jazzy as well (for a current migration, if you choose) by toggling the enable_stamped_cmd_vel: true parameter for each of the Nav2 nodes

@robotpilot
Copy link
Member

Okay! Thanks, @SteveMacenski

@robotpilot robotpilot added the enhancement New feature or request label Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants