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

[SteveMacenski]

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:

• Migrating twist to twiststamped in simulations for recommended default bringups ros-navigation/navigation2#4779
• Migrating twist to twiststamped in simulations for recommended default bringups ros-navigation/docs.nav2.org#614
• Migrating twist to twiststamped in simulations for recommended default bringups ros-navigation/nav2_minimal_turtlebot_simulation#16

[SteveMacenski]

I believe to migrate this package - if wanting to keep in line with Nav2's rolling defaults, the following should be set to true

• teleop_tools/key_teleop/key_teleop/key_teleop.py

  Line 130 in ac63b92

  self._publish_stamped_twist = self.declare_parameter('twist_stamped_enabled', False).value

• The same option from key teleop made for mouse teleop

  teleop_tools/mouse_teleop/mouse_teleop/mouse_teleop.py

  Line 61 in ac63b92

  self._pub_cmd = self.create_publisher(Twist, 'mouse_vel', 10)

Let me know if you want me to open a PR

[christophfroehlich]

Isn't this already done by @bmagyar in #90?

[bmagyar]

Finally!

…

On Tue, 3 Dec 2024, 20:26 Steve Macenski, ***@***.***> wrote: I believe to migrate this package - if wanting to keep in line with Nav2's rolling defaults, the following should be set to true - https://github.com/ros-teleop/teleop_tools/blob/ac63b92477232d8fd73fa9db938ead6735c31971/key_teleop/key_teleop/key_teleop.py#L130 - The same option from key teleop made for mouse teleop https://github.com/ros-teleop/teleop_tools/blob/ac63b92477232d8fd73fa9db938ead6735c31971/mouse_teleop/mouse_teleop/mouse_teleop.py#L61 — Reply to this email directly, view it on GitHub <#93 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AA24PYLVFCK7ABWGGH6WIWL2DYHXJAVCNFSM6AAAAABS6S3VGWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJVGQ4DENBUGE> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>