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

Add airspeed message to uxrce dds topics #23847

Merged
merged 2 commits into from
Nov 5, 2024

Conversation

qtweng
Copy link

@qtweng qtweng commented Oct 23, 2024

Solved Problem

Airspeed message not included in dds_topics.yaml, but is significant for offboard control of fixed wing vehicles.

Fixes #{Github issue ID}

Solution

  • Added airspeed topic to dds_topics.yaml

Test coverage

  • Simulation/hardware testing: built for orange cube plus and verified topic readable by ros2 through uart

@bresch
Copy link
Member

bresch commented Oct 24, 2024

Don't you rather want AirspedValidated?

@qtweng
Copy link
Author

qtweng commented Oct 25, 2024

Yes, will be updating to use airspeed validated, flashing, and verifying.

@qtweng
Copy link
Author

qtweng commented Oct 25, 2024

Verified that it works on cube orange plus

@dakejahl dakejahl enabled auto-merge (squash) November 5, 2024 04:15
@dakejahl dakejahl merged commit e85ffa0 into PX4:release/1.15 Nov 5, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants