DAIRLab / dairlib

MIT License
76 stars 27 forks source link

Invariant ekf #96

Closed Nanda-Kishore-V closed 4 years ago

Nanda-Kishore-V commented 5 years ago

Changes:

mposa commented 5 years ago

Per a face to face discussion with Nanda last week, we weren't sure where the lightly-modified third-party EKF code should be located. The goals are to:

The best option we came up with was to fork Ross Hartley's code, and make changes in that branch (which will make it easy to later merge upstream changes). Include this repository as a pegged external with optional local installation, like we do with Drake.

@yminchen @Nanda-Kishore-V @yangwill Thoughts on this? We wanted to sleep on it, to ensure we aren't adding too much complexity. Having too many of these external repositories might, ultimately, get confusing.

mposa commented 5 years ago

@yminchen @Nanda-Kishore-V @yangwill Following up on this. It'd be good to get this into a form everyone's happy with, and merge it.

yminchen commented 5 years ago

@mposa @Nanda-Kishore-V, @yangwill and I discussed. We think that it's a good idea to fork Ross's code, add our own changes, and use this repo as a pegged external.

Nanda-Kishore-V commented 4 years ago

What's new: invariant-ekf is moved to its own (forked) repository. The default behavior is to use the pegged version of this invariant-ekf repository. However, it can also be downloaded as a local repository for making changes.