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

AP_NavEKF3: fixes in VelPos fusion #29283

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rishabsingh3003
Copy link
Contributor

@rishabsingh3003 rishabsingh3003 commented Feb 12, 2025

This PR attempts the following:

  1. Slight cleanup of VelPosFusion(). Instead of using bool fusePosData and bool fuseVelData, I replace it with a enum to get more information about the source of the sensor being fused. I have also tried to get rid of some of the global variables which don't need to be global.
  2. Allow ExtNav and GPS to work without Vel Z being fused. Currently, if you have VELXY defined in the Sources, the Z component also gets fused irrespective (bug fix). Replaces: AP_NavEKF3: Allow extnav to work without vertical velocity #29213
  3. Use the correct External Nav variances when both GPS and Ext Nav are being used (bug fix) Closes: External navigation measurement noise is overwritten by GPS reported accuracy in EKF #24020.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

External navigation measurement noise is overwritten by GPS reported accuracy in EKF
1 participant