PX4 / PX4-ECL

Estimation & Control Library for Guidance, Navigation and Control Applications
http://dev.px4.io
BSD 3-Clause "New" or "Revised" License
477 stars 508 forks source link

delay time: potentially reduce delay time if airspeed is not used #1017

Closed BazookaJoe1900 closed 3 years ago

BazookaJoe1900 commented 3 years ago

it seems that if some observes are not used, still their delay time parameters are used and might increase the delay buffers.

this pr should check if observes are used and if not, ignore their delay parameters.

  1. airspeed - seems that if param_ekf2_arsp_thr is >0 it can be used
  2. baro - I couldn't find (if) there is parameter that can imply it used or not. so there is still if(??) on the code !
  3. aux velocity - I couldn't find (if) there is parameter that can imply it used or not. so there is still if(??) on the code !
BazookaJoe1900 commented 3 years ago

there is need to add EKF2_ARSP_THR to be be ParamExtFloat here:

https://github.com/PX4/PX4-Autopilot/blob/a728431e4350917f3add9814732715101ac99b4d/src/modules/ekf2/EKF2.hpp#L459

so after @priseborough will look at that too, I will do the change there too

Edit - As you mention above