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

Make EKF more Generic and add TOML file for Handling Dependencies #301

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

chacalnoir
Copy link

  • The original EKF assumes a linear, time invariant (LTI) transition matrix. That is, in general, not a requirement for an EKF. Since the measurement Jacobian can be overridden, the transition matrix should also be able to be computed at each step. This PR adds an override to enable that capability.
  • The original install method was via setup.py only. This doesn't work with newer build systems that are using TOML files, as the dependencies don't seem to be handled correctly. This PR updates the setup.py file to be a TOML file instead to correctly handle dependencies.
  • The README docs mentioned that Python 2.7 and 3.4 support was supposed to be dropped around 2018. The setup.py file doesn't show this. As part of updating to the TOML file, the classifiers were also updated to be Python 3.7.
  • As part of the TOML file, I did add in flake8 and autopep definitions. They are not being used to lint currently, but provide the basis for linting if desired.

@nbarascud-sc
Copy link

@rlabbe Any interest in merging this ?

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

Successfully merging this pull request may close these issues.

None yet

2 participants