-
Notifications
You must be signed in to change notification settings - Fork 100
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
Allow a subset of <geometry>
instead of <inertia>
#176
Comments
Original comment by Ian Chen (Bitbucket: Ian Chen, GitHub: iche033). I remember there was talk about changing the default inertia value (from identity matrix) to be something computed on geometry type, mass, and size. So if the |
Original comment by Kunal Tyagi (Bitbucket: kunaltyagi). Has there been a progress on either issue? I would like to contribute to this coz it is a regular problematic issue for my team. |
we also discussed this a bit in #95 (comment) |
Original report (archived issue) by Shane Loretz (Bitbucket: Shane Loretz, GitHub: sloretz).
Issue
The
<inertial>
element requires an inertial matrix<inertia>
. When creating models I often find myself plugging the geometry of the<collision>
into one of the formulas on this wikipedia page, which can be prone to errors as I often end up changing the geometry.Proposal
The proposal is to allow a subset of the shapes in
<geometry>
(box, cylinder, sphere) as an alternative to<inertia>
. The inertia matrix values would be calculated by assuming uniform density.Related
Issue #95 combined with this feature would make it much easier to specify the inertia on a link with multiple
<collision>
.The text was updated successfully, but these errors were encountered: