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

[Estimation] Dynamic reset functionality for ExtendedKalmanFilter #553

Merged
merged 2 commits into from
Aug 25, 2019
Merged

[Estimation] Dynamic reset functionality for ExtendedKalmanFilter #553

merged 2 commits into from
Aug 25, 2019

Conversation

prashanthr05
Copy link
Contributor

This PR adds a method to dynamically reset the EKF components through a function call.
Also, includes a few minor changes

  • added a ignore function template to ignore unused parameters
  • added an ekfComputeJacobian method to also take into account inputs during computation
  • removed stray variable

@prashanthr05
Copy link
Contributor Author

@traversaro if you could review this PR, it would be great. Thank you!

@prashanthr05
Copy link
Contributor Author

@traversaro I've addressed your review in the squashed commit.

@prashanthr05
Copy link
Contributor Author

@traversaro Travis seems to have some timeout issues. Could you check it and let me know if I should do something about it?

@prashanthr05
Copy link
Contributor Author

@traversaro friendly ping! :)

@traversaro
Copy link
Member

I restarted the Travis build, in the meanwhile can you resolve the conflicts on the release notes? Btw, I think you should have the powers to restart Travis jobs, let me know if you do not have them.

@prashanthr05
Copy link
Contributor Author

I restarted the Travis build, in the meanwhile can you resolve the conflicts on the release notes?

Done in 4e3680d. Thanks!

@prashanthr05
Copy link
Contributor Author

Btw, I think you should have the powers to restart Travis jobs, let me know if you do not have them.

I think I have them. I see a Cancel Build option on a currently running job.

@traversaro traversaro merged commit 34d9c09 into robotology:devel Aug 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@traversaro traversaro traversaro approved these changes

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

Successfully merging this pull request may close these issues.
2 participants
@prashanthr05 @traversaro