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

Move gazebo::physics::Inertial to here #32

Closed
osrf-migration opened this issue Nov 3, 2015 · 5 comments
Closed

Move gazebo::physics::Inertial to here #32

osrf-migration opened this issue Nov 3, 2015 · 5 comments
Labels
enhancement New feature or request

Comments

@osrf-migration
Copy link

Original report (archived issue) by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).


There's mostly math in the gazebo::physics::Inertial class. It would be convenient to move it to ignition-math so that it could be used without linking to the physics library.

@osrf-migration
Copy link
Author

Original comment by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).


pull request #112

@osrf-migration
Copy link
Author

Original comment by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).


addition operator in pull request #115

@osrf-migration
Copy link
Author

Original comment by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).


I've thought about adding a subtraction operator, but the order of operations could be tricky if you subtract multiple small Inertials from a larger Inertial. I'm going to hold off on this for now.

@osrf-migration
Copy link
Author

Original comment by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).


One last feature I'd like to add is the ability to set the rotation of the inertia pose or of the MassMatrix without affecting the moment of inertia matrix in the base frame (ie. the output of MOI()). This is related to this test-case.

@osrf-migration
Copy link
Author

Original comment by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).


  • changed state from "new" to "resolved"

MOI rotation added in pull request #121, that should be all the features

@osrf-migration osrf-migration added major enhancement New feature or request labels Apr 15, 2020
scpeters added a commit that referenced this issue Feb 15, 2022
Signed-off-by: Steve Peters <[email protected]>
scpeters added a commit that referenced this issue Feb 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant