Skip to content

Conversation

@qtweng
Copy link
Contributor

@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
Contributor Author

qtweng commented Oct 25, 2024

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

@qtweng
Copy link
Contributor 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