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

Improve EV height starting condition #970

Open
bresch opened this issue Feb 2, 2021 · 1 comment
Open

Improve EV height starting condition #970

bresch opened this issue Feb 2, 2021 · 1 comment

Comments

@bresch
Copy link
Member

bresch commented Feb 2, 2021

follows PX4/PX4-Autopilot#16489

The EV height starting condition only checks if the gathered data is not too old :

if (!_control_status.flags.ev_hgt && isRecent(_time_last_ext_vision, 2 * EV_MAX_INTERVAL)) {
)
In some cases, it can be that the EV system provides wrong data; in that case, it should be rejected and EV height should not start (or re-start).

FYI @cimbar

@antonio-sc66
Copy link

Checking the Z variance should help (posVar(2) of the extVisionSample struct) as one of the common problems when the EV systems are providing wrong data is that the position or velocity values tend to vary a lot and rapidly. The problem with this method is that there are EV systems that don't fill the covariance matrix, in those cases the posVar will have the EKF2_EVP_NOISE and the filtering wont apply.

https://github.com/PX4/PX4-Autopilot/blob/f13c3a7d44c58f69c77f9e4f82cf3d6a41ec9551/src/modules/ekf2/EKF2.cpp#L1294-L1303

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

No branches or pull requests

3 participants