Closed wxmerkt closed 1 year ago
Also, I think this is a good thing to update this inertia, but I'm afraid this might break benchmarks or other reproducible tests relying on the current values. But I guess we'll only have to advertise this correctly in release notes.
I don't think we need to worry about this: We haven't used it in benchmarks yet to my knowledge - and couldn't due to the zero inertials - i.e. that was the reason behind #122 and the original issue upstream in Crocoddyl about failed optimisation (due to zero inertials).
Fixes #122