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.

Sign up for GitHub

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

Jump to bottom

EKF_sym class rewritten to c++ #9

Merged
merged 33 commits into from Apr 8, 2021
Merged

EKF_sym class rewritten to c++ #9

merged 33 commits into from Apr 8, 2021

Conversation

jwooning
Copy link
Contributor

@jwooning jwooning commented Mar 26, 2021 •
edited

Work in progress

  • Currenlty only functions for live_kf, dynamic loading of specific implementation functions is still required
  • Not all functions like rewinding are done

pd0wm
pd0wm reviewed Apr 7, 2021
View reviewed changes
rednose/helpers/ekf_sym.cc Outdated Show resolved Hide resolved
pd0wm
pd0wm reviewed Apr 7, 2021
View reviewed changes
rednose/helpers/ekf_sym.cc Outdated Show resolved Hide resolved
@pd0wm pd0wm merged commit a0eb4d2 into master Apr 8, 2021
@pd0wm pd0wm deleted the ekfsym-cpp branch April 8, 2021 11:31
@pd0wm pd0wm mentioned this pull request Apr 8, 2021
Closed
rav4kumar added a commit to arneschwarck/rednose that referenced this pull request Apr 9, 2021
@rav4kumar
Merge pull request #1 from commaai/master
3552c27 
EKF_sym class rewritten to c++ (commaai#9)
@wolterhv
Copy link

#12 is a draft PR (WIP) for finishing the rewrite of EKSym functions mentioned by @jwooning on this PR's opening comment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pd0wm pd0wm pd0wm left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jwooning @wolterhv @pd0wm