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

Scripting example to provide altitude callouts in feet defaulting to 400ft warnings #28426

Merged
merged 1 commit into from
Nov 23, 2024

Conversation

timtuxworth
Copy link
Contributor

@timtuxworth timtuxworth commented Oct 17, 2024

Replaces #23368

So many countries have a limit of 400 feet when flying RPAS/UAV (rc planes). This script provides groundstation messages which can be "called out" by QGroundControl, Mission Planner. etc. to advise the pilot what their current altitude is, warn them when approaching 400 feet (or any other set limit) and call out when they have exceeded the limit.

This may not be required on an OpenTX/EdgeTX radio with something like Crossfire/Express LRS as the radio can be configured to do callouts and Yaapu telemetry has a max altitude alert, but EdgeTX won't provide ongoing altitude updates the way this script can do, and this will work with any controller, not just EdgeTX/Yaapu.

After @peterbarker challenged me on this one, I rebased and then had to clean up/improve the example to reflect the code I'm flying now which has changed a bit since the original PR.

Hope it's still ok to merge Peter.

@timtuxworth
Copy link
Contributor Author

I messed up GitHub somehow @peterbarker so I've created this new PR for the altitude callouts, I also renamed the script and updated it based on the one I'm actually flying.

Copy link
Contributor

@peterbarker peterbarker left a comment

Choose a reason for hiding this comment

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

scripting linter detected a problem which needs fixing

@timtuxworth
Copy link
Contributor Author

scripting linter detected a problem which needs fixing

Looks like it's fixed now @peterbarker

@peterbarker peterbarker merged commit 1b4ac6a into ArduPilot:master Nov 23, 2024
100 checks passed
@peterbarker
Copy link
Contributor

Merged it - thanks!

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.

2 participants