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

AP_DDS: Add support for geometry_msgs/TwistStamped velocity publishing from AP #23279

Closed
Ryanf55 opened this issue Mar 22, 2023 · 2 comments · Fixed by #23541
Closed

AP_DDS: Add support for geometry_msgs/TwistStamped velocity publishing from AP #23279

Ryanf55 opened this issue Mar 22, 2023 · 2 comments · Fixed by #23541
Assignees
Labels

Comments

@Ryanf55
Copy link
Collaborator

Ryanf55 commented Mar 22, 2023

Add support for publishing the velocity of the vehicle in the local coordinate system. This message goes hand in hand with #23278 (they are published at the same rate at the same time).

The Twist message captures the linear and angular velocities of the vehicle following ROS REP 103 conventions (SI units, ENU, right-handed quaternion orientation)

@Ryanf55 Ryanf55 moved this to 🆕 New in DDS/ROS2 Mar 22, 2023
@Ryanf55 Ryanf55 added this to DDS/ROS2 Mar 22, 2023
@Ryanf55 Ryanf55 added the ROS label Mar 22, 2023
@Ryanf55 Ryanf55 moved this from 🆕 New to 📋 Backlog in DDS/ROS2 Apr 5, 2023
@Ryanf55 Ryanf55 changed the title AP_DDS: Add support for geometry_msgs/TwistStamped AP_DDS: Add support for geometry_msgs/TwistStamped velocity publishing from AP Apr 13, 2023
@pedro-fuoco
Copy link
Contributor

I can have this one. I will work on it as soon as #23480 merges, as they're quite similar

@Ryanf55 Ryanf55 moved this from 📋 Backlog to 🔖 Ready in DDS/ROS2 Apr 14, 2023
@pedro-fuoco
Copy link
Contributor

Should we use a body-fixed or local NED orientation for this topic ?
MAVROS uses a body-fixed orientation in their local position plugin, and I agree with them, it is more convenient in velocity data to have an x-forward standard

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants