-
Notifications
You must be signed in to change notification settings - Fork 288
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
Inertia class #234
Comments
Inertia class is added in #369. |
I didn't close this yet, because we currently don't have an API that handles the transformation of the spatial inertia matrices. I don't think it would be hard to implement, I just haven't nailed down an API design that seems ideal to me yet. I have two ideas so far. Let's say you want to transform from (1) Take in both frames as arguments:
(2) Or just take in a single transform and assume that the user is passing in the correct transform:
I'm not sure which would be preferable, but I'm leaning towards the latter. I'm also open to alternatives. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions. |
BodyNode
has multiple functionalities and it makes the API complex.Inertia
will take mass, moment of inertia, and center of mass properties fromBodyNode
.BodyNode
Inertia
Inertia
Inertia
Inertia
The text was updated successfully, but these errors were encountered: