Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

EKF: allow initialising without baro depending on configuration #996

Closed
wants to merge 1 commit into from

Conversation

dagar
Copy link
Member

@dagar dagar commented Mar 30, 2021

Attempted followup to #931.

  • every potential height source maintains a filtered value at the delayed time horizon
  • in Ekf::initialiseFilter() only the configured height source is used, but valid height data is required at init time
  • height resets use filtered delayed time horizon data rather than most recent measurement

EKF/estimator_interface.h Outdated Show resolved Hide resolved
@dagar dagar force-pushed the pr-ekf_init_no_baro branch from 16f6385 to 5131204 Compare March 31, 2021 14:25
@priseborough
Copy link
Collaborator

Can we do a test where the board is started without the baro driver?

@dagar dagar force-pushed the pr-ekf_init_no_baro branch 4 times, most recently from f211e93 to 14474fb Compare April 8, 2021 14:21
@dagar dagar force-pushed the pr-ekf_init_no_baro branch from 14474fb to 682805d Compare July 9, 2021 20:29
@dagar dagar force-pushed the pr-ekf_init_no_baro branch from 682805d to a5452e1 Compare July 9, 2021 20:46
@priseborough
Copy link
Collaborator

There's a lot of code here for the ability to initialise height without a baro. Can we split this into one PR into what is required for initialisation without baro and a second PR with other improvements, eg median filtering. For example we could initialise height to a fake measurement of zero whilst awaiting the selected height sensor to pass checks.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants