ros_gz icon indicating copy to clipboard operation
ros_gz copied to clipboard

Add mappings for gz.msgs.AirSpeedSensor and gz.msgs.Altimeter

Open srmainwaring opened this issue 1 year ago • 6 comments

Desired behavior

Provide mappings for sensor data:

  • gz.msgs.AirSpeedSensor
  • gz.msgs.Altimeter

Alternatives considered

  • NA.

Implementation suggestion

The main issue is which ROS message type to target as these messages do not have a corresponding message in sensor_msgs.

Additional context

Would be happy to submit a PR for this feature request once a suitable ROS message type is agree upon.

srmainwaring avatar Apr 19 '23 12:04 srmainwaring

We've been creating our own ROS messages when a compatible message type is not available in the ROS ecosystem (see https://github.com/gazebosim/ros_gz/tree/ros2/ros_gz_interfaces). Can we do the same here?

azeey avatar May 01 '23 21:05 azeey

Feel free to add me as a reviewer @srmainwaring

ahcorde avatar May 08 '23 22:05 ahcorde

Adding Altimeter bridging here : https://github.com/gazebosim/ros_gz/pull/413. Note, addin AirSpeedSensor also requires adding SensorNoise msg.

adityapande-1995 avatar Jun 13 '23 22:06 adityapande-1995

SensorNoise msg work in progress : https://github.com/gazebosim/ros_gz/pull/417

adityapande-1995 avatar Jun 14 '23 22:06 adityapande-1995

Is there anything missing for the AirSpeedSensor? I would also be very interested in having this data in ROS

wittenator avatar Nov 09 '23 10:11 wittenator

Just for the sake of documentation: It seems that the Air Speed sensor msgs only exists in gz_msgs 9+. Since the current stable version of ros_gz is using Fortress and therefore gz_msgs version 7, there is no message to bridge atm.

wittenator avatar Nov 13 '23 13:11 wittenator