-
Notifications
You must be signed in to change notification settings - Fork 454
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
added center of mass to Panda DH definition (and cleaned it up a bit) #304
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was about to suggest the same thing. However, it must be noted that the parameters come from https://github.com/marcocognetti/FrankaEmikaPandaDynModel which uses the axes as in https://frankaemika.github.io/docs/control_parameters.html, but the model here ignores the (7) axes and links the last solid to the (F) axes. Therefore, the center of mass of the last solid must be given in (F) coordinates.
Co-authored-by: Anonymous Researcher <[email protected]>
@AunSiro Thanks! Indeed, both the marcocognetti and frankaemika DH definition include a static 8th joint for the flange, while here this is rolled into the 7th joint, so the center of mass needs to be updated. |
should now be compatible with the latest version in master |
I too was about to suggest the same thing, before I saw your pull request. It would be great to see this merged! |
In pull request #294 some of the inertia parameters were ported from the matlab version. The center of mass for the links was still missing (without which dynamics calculations will be wrong). I've added them here and also cleaned up the white space a bit.