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

Re-assess barometer weight and logic for altitude fusion #4692

Closed
digitalentity opened this issue May 8, 2019 · 1 comment
Closed

Re-assess barometer weight and logic for altitude fusion #4692

digitalentity opened this issue May 8, 2019 · 1 comment
Milestone

Comments

@digitalentity
Copy link
Member

When accelerometer weight is reduced by #4681 or manually, barometer influence is not enough to maintain altitude consistently and reliably.

TODO:

  • Re-evaluate barometer weight and find a better default
  • Re-evaluate barometer fusion into altitude (possibly extract climb rate from the raw data and correct velocity from that data separately).
@shota3527
Copy link
Contributor

@digitalentity
Is #4681 a good fix for the problem?
It is not reducing the accelerometer weight but set the measured acceleration to 0. Which means velocity/position estimation by accelerometer is also 0. Then value from accelerometer and baro is fighting with earth other.

@DzikuVx DzikuVx closed this as completed Feb 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants