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

Add the possibility to use manif objects in the ForwardEuler integrator #379

Merged
merged 5 commits into from
Jul 23, 2021

Conversation

GiulioRomualdi
Copy link
Member

This PR introduces the possibility to use ForwardEuler integrator with manif objects. Thanks to this I removed the Baumgarte stabilization in FloatingBaseSystemKinematics class

@GiulioRomualdi GiulioRomualdi self-assigned this Jul 23, 2021
@GiulioRomualdi GiulioRomualdi changed the title Add the possibility to use manif object in the ForwardEuler integrator Add the possibility to use manif objects in the ForwardEuler integrator Jul 23, 2021
@prashanthr05
Copy link
Collaborator

prashanthr05 commented Jul 23, 2021

In the need for extension towards also right trivialization,

Maybe you could allow the user to set the trivialization (enum class) as done here and use the corresponding left/right composition accordingly, as done in an implementation of RKMK4 Liegroup integrator, here.

warning: associated links are pointed to private repositories, only dic-iit organization members have acess.

@GiulioRomualdi GiulioRomualdi force-pushed the ContinuousDynamicalSystem/manif branch from b13acc5 to 82cde16 Compare July 23, 2021 19:51
@GiulioRomualdi GiulioRomualdi merged commit ead6a6a into master Jul 23, 2021
@GiulioRomualdi GiulioRomualdi deleted the ContinuousDynamicalSystem/manif branch July 23, 2021 21:33
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.

4 participants