DISCOWER / PX4-Space-Systems

PX4 Space Systems Fork
BSD 3-Clause "New" or "Revised" License
2 stars 1 forks source link

EKF Dynamics Update #2

Open Pedro-Roque opened 9 months ago

Pedro-Roque commented 9 months ago

Solved Problem

Attempt at fixing the EKF for microgravity dynamics.

Solution

For now purely deleting old solution, but can be made to accommodate in parallel.

Test coverage

Pedro-Roque commented 9 months ago

For now, I chose to ignore merging-compatible modifications with PX4 main tree - but it could easily be done in this case. Just checking if this is the direction in which we want to go.

Still requires updates on the EKF state (we don't need or want wind estimates, for instance)

Jaeyoung-Lim commented 9 months ago

For now, I chose to ignore merging-compatible modifications with PX4 main tree - but it could easily be done in this case. Just checking if this is the direction in which we want to go.

Probably a copy-paste on a new module?

Still requires updates on the EKF state (we don't need or want wind estimates, for instance)

This is configurable in upstream by https://github.com/PX4/PX4-Autopilot/pull/22172, just FYI

Pedro-Roque commented 9 months ago

For now, I chose to ignore merging-compatible modifications with PX4 main tree - but it could easily be done in this case. Just checking if this is the direction in which we want to go.

Probably a copy-paste on a new module?

Still requires updates on the EKF state (we don't need or want wind estimates, for instance)

This is configurable in upstream by PX4#22172, just FYI

Then perhaps we could even make a configuration to which dynamics we should use. It would make it more functional perhaps.