Skip to content

4.8:33792/zero-takeoff-alt-check - #7971

Open
Hwurzburg wants to merge 1 commit into
ArduPilot:masterfrom
Hwurzburg:4.8/33792/zero-takeoff-alt-check
Open

4.8:33792/zero-takeoff-alt-check#7971
Hwurzburg wants to merge 1 commit into
ArduPilot:masterfrom
Hwurzburg:4.8/33792/zero-takeoff-alt-check

Conversation

@Hwurzburg

Copy link
Copy Markdown
Contributor

Documents the zero-takeoff-altitude mission arming check added in ArduPilot/ardupilot#33792.

AP_Arming::mission_checks() now fails arming with "Mission: Zero takeoff altitude" if any takeoff-type mission command has an altitude of exactly 0 (negative altitudes aren't checked). Added to both the Mission: prose subsection and the main "Pre-Arm Failure Messages (All vehicle types)" table, matching how the other Mission: entries are each mirrored in the table.

@Hwurzburg
Hwurzburg force-pushed the 4.8/33792/zero-takeoff-alt-check branch from 563db2b to e774ffc Compare August 16, 2026 15:35
@Hwurzburg
Hwurzburg requested a review from IamPete1 August 18, 2026 11:33
IamPete1
IamPete1 previously approved these changes Aug 18, 2026

@IamPete1 IamPete1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks

document zero takeoff altitude mission check

Added in ArduPilot/ardupilot#33792. AP_Arming::mission_checks() now
fails arming with "Mission: Zero takeoff altitude" if any takeoff-type
mission command has an altitude of exactly 0, since that's almost
always a forgotten value rather than intentional. Negative altitudes
are not checked.

Added to both the Mission: prose subsection and the main "Pre-Arm
Failure Messages (All vehicle types)" table, matching how the other
Mission: entries (No mission library present, No rally library
present, Missing mission item: xxxx) are each mirrored in the table.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
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.

2 participants