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

Use //link/inertial/density element in parser #1329

Closed
scpeters opened this issue Sep 27, 2023 · 1 comment · Fixed by #1335
Closed

Use //link/inertial/density element in parser #1329

scpeters opened this issue Sep 27, 2023 · 1 comment · Fixed by #1335
Labels
enhancement New feature or request 🎵 harmonic Gazebo Harmonic

Comments

@scpeters
Copy link
Member

scpeters commented Sep 27, 2023

Desired behavior

After describing the //collision/density element, the Proposal for Automatic Moments of Inertia Calculations states: "A //link/inertial/density element would also be added in the spec to allow users to specify the density values on a link level instead of specifying the same values for each collision."

This element should be was added to the spec in #1299 but still needs to be used in the parser.

Alternatives considered

Implementation suggestion

Additional context

@scpeters scpeters added enhancement New feature or request 🎵 harmonic Gazebo Harmonic labels Sep 27, 2023
@scpeters scpeters changed the title Add //link/inertial/density element Use //link/inertial/density element in parser Sep 27, 2023
@scpeters scpeters moved this from Inbox to In review in Core development Oct 8, 2023
@scpeters
Copy link
Member Author

scpeters commented Oct 8, 2023

see #1335

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request 🎵 harmonic Gazebo Harmonic
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant