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

ekf2: Split static pressure correction into separate RH and LH ellipsoids #11545

Merged
merged 3 commits into from
Mar 5, 2019

Conversation

priseborough
Copy link
Contributor

@priseborough priseborough commented Feb 26, 2019

Splits ellipsoid used to correct for static source position error into separate
RH and LH regions. This is required where a pixhawk 'cube' autopilot is externally mounted with the usb port facing left or right causing a rise or fall in internal pressure with RH vs LH movement.

@priseborough priseborough changed the title ekf2: Split static pressure correction into separate RH ad LH ellipsoids ekf2: Split static pressure correction into separate RH and LH ellipsoids Feb 26, 2019
@priseborough
Copy link
Contributor Author

@thomasgubler FYI

@lamping7
Copy link
Member

Don't get mad at Jenkins... Improvements to the HITL testing platform suite by @dagar made a change in the Jenkinsfile... Rebase.

@thomasgubler
Copy link
Contributor

diff looks good to me

@thomasgubler
Copy link
Contributor

@bresch

bresch
bresch previously approved these changes Feb 27, 2019
Copy link
Member

@bresch bresch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

Copy link
Member

@bkueng bkueng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @priseborough.

src/modules/ekf2/ekf2_main.cpp Outdated Show resolved Hide resolved
src/modules/ekf2/ekf2_params.c Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants