-
Notifications
You must be signed in to change notification settings - Fork 148
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
Add mappings for gz.msgs.AirSpeedSensor and gz.msgs.Altimeter #382
Comments
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? |
Feel free to add me as a reviewer @srmainwaring |
Adding Altimeter bridging here : #413. |
SensorNoise msg work in progress : #417 |
Is there anything missing for the AirSpeedSensor? I would also be very interested in having this data in ROS |
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. |
Desired behavior
Provide mappings for sensor data:
gz.msgs.AirSpeedSensor
gz.msgs.Altimeter
Alternatives considered
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.
The text was updated successfully, but these errors were encountered: