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.

Sign up for GitHub

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

Jump to bottom

ekf2: increase default baro noise 2 -> 3.5 m #14986

Merged
merged 1 commit into from
May 28, 2020
Merged

ekf2: increase default baro noise 2 -> 3.5 m #14986

merged 1 commit into from
May 28, 2020

Conversation

dagar
Copy link
Member

@dagar dagar commented May 27, 2020

As discussed on the dev call.

FYI @priseborough

@dagar dagar added the EKF2 label May 27, 2020
@dagar dagar added this to the Release v1.11.0 milestone May 27, 2020
@dagar dagar requested review from RomanBapst and bresch May 27, 2020 15:34
@dagar dagar merged commit 0ec48cf into master May 28, 2020
@dagar dagar deleted the pr-ekf2_baro_noise_default branch May 28, 2020 12:35
@RomanBapst
Copy link
Contributor

@dagar We should also lower the limit for the gps vertical velocity observation variance.
See #14640

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LorenzMeier LorenzMeier LorenzMeier approved these changes

@RomanBapst RomanBapst Awaiting requested review from RomanBapst

@bresch bresch Awaiting requested review from bresch

Assignees
No one assigned
Labels
EKF2
Projects
None yet
Milestone
Release v1.11.0
Development

Successfully merging this pull request may close these issues.
3 participants
@dagar @RomanBapst @LorenzMeier