Fix APM takeoff when vehicle is already armed and on the ground - #15063
eris-ismajli wants to merge 2 commits into
Conversation
Codecov Report❌ Patch coverage is
❌ Your patch check has failed because the patch coverage (0.00%) is below the target coverage (30.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## master #15063 +/- ##
==========================================
+ Coverage 25.47% 34.82% +9.35%
==========================================
Files 769 950 +181
Lines 65912 85902 +19990
Branches 30495 39594 +9099
==========================================
+ Hits 16788 29917 +13129
+ Misses 37285 36015 -1270
- Partials 11839 19970 +8131
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 682 files with indirect coverage changes Continue to review full report in Codecov by Harness.
🚀 New features to boost your workflow:
|
Build ResultsPlatform Status
All builds passed. Pre-commit
Pre-commit hooks: 0 passed, 0 failed, 0 skipped. Test Resultslinux-coverage-integration: 52 passed, 0 skipped Code CoverageCoverage: 71.2% No baseline available for comparison Artifact Sizes
Updated: 2026-09-09 15:01:36 UTC • Commit: bc292b4 • Triggered by: Linux |
Description
Fixes an issue where APM takeoff does nothing if the vehicle is already armed while on the ground.
Previously,
APMFirmwarePlugin::startTakeoff()only switched to Takeoff mode and attempted to arm the vehicle when it was disarmed. If the vehicle was already armed, the takeoff sequence was skipped entirely.This change ensures that:
Type of Change
Testing
Platforms Tested
Flight Stacks Tested
Screenshots
Checklist
Related Issues
By submitting this pull request, I confirm that my contribution is made under the terms of the project's dual license (Apache 2.0 and GPL v3).