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

WIP: AP_Airspeed: Blending of several sensors V2 #7891

Closed
wants to merge 2 commits into from

Conversation

EShamaev
Copy link
Member

@EShamaev EShamaev commented Mar 10, 2018

Second version based on issues raised by @WickedShell

Main changes are:

  • minimum EKF speed to enable health determination
  • good/bad delay to prevent erratic behaviour

@EShamaev
Copy link
Member Author

image

Takeoff and flight with 15 m/s wind, changing wind 15->5->15, changing directions 0->180->0, turbulence up to 5.

00000041.zip

@EShamaev
Copy link
Member Author

@priseborough Can you please clarify why wind fusion (and airspeed from sensors as well) is disabled while we are on the ground?
NavEKF2_core::setWindMagStateLearningMode() in control.cpp

@priseborough
Copy link
Contributor

Because air speed measurements on ground are highly unreliable due to unknown orientation wrt wind. Also zero sideslip assumption cannot be used.

@EShamaev EShamaev force-pushed the AP_Airspeed_Blending_2 branch 2 times, most recently from 4248c81 to 519a3e4 Compare November 11, 2018 13:02
@OXINARF
Copy link
Member

OXINARF commented Nov 16, 2018

Needs to be rebased on master and have submodule changes removed.

@EShamaev
Copy link
Member Author

Not only rebased. Due to "recent" changes, I will update this completely.

@EShamaev EShamaev added the WIP label Nov 16, 2018
@EShamaev EShamaev changed the title AP_Airspeed: Blending of several sensors V2 WIP: AP_Airspeed: Blending of several sensors V2 Nov 16, 2018
@EShamaev EShamaev force-pushed the AP_Airspeed_Blending_2 branch 2 times, most recently from ee47e72 to 53f6eb6 Compare November 18, 2018 06:42
@IamPete1
Copy link
Member

@EShamaev We now have EKF affinity for airspeed, is this PR something you would still like to take forward?

@davidbuzz
Copy link
Collaborator

davidbuzz commented Aug 17, 2021

superceded by ekf lanes/affinity.

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.

None yet

5 participants