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_Soaring: Add named value float publisher of soaring thermal EKF state #29030

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

Ryanf55
Copy link
Collaborator

@Ryanf55 Ryanf55 commented Jan 9, 2025

Purpose

  • This sends the EKF estimate of the thermal to mavproxy_soar so a dev can visualize where AP thinks the thermal is and how big it is
  • Useful until we have proper mavlink packets for soaring state
  • I threw in the strength, which I may scale the line thickness on color it different colors depending on strength like the thermal estimators on flight instruments do. The strength is not yet used in mavproxy.

Question

Should they be throttled to something like 4HZ? Edit: Yea, on a real vehicle, we should throttle this.

Demo

See ArduPilot/MAVProxy#1500

Make sure to define HAL_SOARING_NVF_EKF_ENABLED to 1

* This sends the EKF estimate of the thermal to mavproxy_soar
* Useful until we have proper mavlink packets for soaring state

Signed-off-by: Ryan Friedman <[email protected]>
@Ryanf55 Ryanf55 changed the title AP_Soaring: Add named value float publisher AP_Soaring: Add named value float publisher of thermal EKF state Jan 9, 2025
@Ryanf55 Ryanf55 changed the title AP_Soaring: Add named value float publisher of thermal EKF state AP_Soaring: Add named value float publisher of soaring thermal EKF state Jan 14, 2025
@tridge tridge merged commit fa04004 into ArduPilot:master Jan 14, 2025
99 checks passed
@Ryanf55 Ryanf55 deleted the soaring-nvf-ekf branch January 14, 2025 00:57
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