Skip to content

AP_Scripting: added qnh_alt example #29705

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

Merged
merged 1 commit into from
Apr 9, 2025
Merged

Conversation

tridge
Copy link
Contributor

@tridge tridge commented Apr 6, 2025

useful for BVLOS ops. The one in AFS is only available if AFS is enabled

@tridge tridge requested a review from peterbarker April 6, 2025 08:19
Copy link
Contributor

@timtuxworth timtuxworth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice clear example of sending a named float. A couple of suggestions re comments and a typo (in a comment).

--[[
example that sends QNH altitude to GCS as a NAMED_VALUE_FLOAT QNG_ALT_FT in feet

Note: operator must be QNH_PRESSURE in hPa before each flight!
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

must "set"

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/be "Note: operator must set QNH_PRESSURE in hPa before each flight!"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@@ -0,0 +1,51 @@
--[[
example that sends QNH altitude to GCS as a NAMED_VALUE_FLOAT QNG_ALT_FT in feet
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
example that sends QNH altitude to GCS as a NAMED_VALUE_FLOAT QNG_ALT_FT in feet
example that sends QNH altitude to GCS as a NAMED_VALUE_FLOAT QNH_ALT_FT in feet

@tridge tridge merged commit bd7e01d into ArduPilot:master Apr 9, 2025
121 of 127 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants