This repository has been archived by the owner on May 1, 2024. It is now read-only.
EKF Tuning Update #36
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Set conservative defaults as a baseline for tuning Add a missing parameter for magnetometer observation noise. Correct error in definition of magnetic heading observations noise (previous parameter defined the variance directly, not the noise). Scale vertical GPS noise to allow for average EPV to EPH ratio. Update documentation and display names for consistency.
Fix bug where noise parameters were not being squared when calculating the observation variance. Use GPS reported speed accuracy to set velocity observation noise and use the parameter to set the minimum.
A larger position uncertainty is required when flying without GPS to reduce tilt attitude estimation errors caused by vehicle manoeuvring. This needs to be tuneable to allow optimisation for different use cases (e.g. outdoor vs indoor).
This was referenced Feb 3, 2016
|
@priseborough Reviewing this! |
|
@priseborough It requires proper indentation, can you run this code through code check script in PX4. |
|
OK |
|
@priseborough Looks good, merging! |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Flight testing has been completed (see following figures). The sensor error estimates are now more reactive which improves robustness at he expense of accuracy.