Fly View: add altitude support for Click to ROI#14438
Conversation
|
Thanks for your first pull request! 🎉 A maintainer will review this soon. Please ensure:
We appreciate your contribution to QGroundControl! |
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 #14438 +/- ##
==========================================
+ Coverage 25.47% 26.55% +1.08%
==========================================
Files 769 767 -2
Lines 65912 66291 +379
Branches 30495 30669 +174
==========================================
+ Hits 16788 17605 +817
+ Misses 37285 36190 -1095
- Partials 11839 12496 +657
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 125 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
Build ResultsPlatform Status
All builds passed. Pre-commit
Pre-commit hooks: 2 passed, 42 failed, 7 skipped. Test Resultslinux-coverage: 88 passed, 0 skipped Code Coverage
Artifact Sizes
Updated: 2026-05-27 13:34:46 UTC • Triggered by: Android |
Description
Adds altitude support for Fly View "Click to ROI".
Previously, ROI commands only sent latitude and longitude coordinates,
causing the altitude field to remain unset/zero. This update includes
altitude in the ROI coordinate sent through MAVLink.
This improves camera/gimbal targeting accuracy when using Click to ROI.
Fixes #14394.
Type of Change
Testing
Platforms Tested
Flight Stacks Tested
Screenshots
N/A
Checklist
Related Issues
#14394