Repository navigation
fix(Vehicle): use standard PX4 airspeed calibration value (5.1) - #14781
Conversation
MAV_CMD_PREFLIGHT_CALIBRATION param6 = 2 is the standard airspeed calibration value. PX4 deprecated 1 in Feb 2017 but still accepts it; 2 matches the MAVLink spec and PX4's own commander CLI usage. Replaces mavlink#14774. Thanks to @tustin-dev for the original fix.
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 @@
## Stable_V5.1 #14781 +/- ##
==============================================
Coverage ? 32.61%
==============================================
Files ? 784
Lines ? 67564
Branches ? 31292
==============================================
Hits ? 22035
Misses ? 30666
Partials ? 14863
Flags with carried forward coverage won't be shown. Click here to find out more.
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: 37 passed, 0 skipped Code Coverage
Artifact Sizes
Updated: 2026-08-08 16:58:05 UTC • Commit: 735e982 • Triggered by: Android |
Stable_V5.1 version of #14780, replacing #14774 — thanks to @tustin-dev for identifying this and submitting the original fix!
Summary
Use the standard MAVLink value
2for airspeed calibration inMAV_CMD_PREFLIGHT_CALIBRATIONparam6. PX4 deprecated1in Feb 2017 but still accepts both, so this is a spec-compliance cleanup:2matches the MAVLink spec and what PX4's owncommander calibrate airspeedCLI sends.The unit test from #14774 was dropped — see #14780 for the rationale.