-
Notifications
You must be signed in to change notification settings - Fork 269
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
Hydrodynamic added mass #1462
Labels
enhancement
New feature or request
MBARI buoy
Sponsored by MBARI buoy sim project: https://github.com/osrf/buoy_sim
physics
Involves Ignition Physics
Comments
chapulina
added
enhancement
New feature or request
MBARI buoy
Sponsored by MBARI buoy sim project: https://github.com/osrf/buoy_sim
labels
Apr 27, 2022
|
This was referenced Jun 29, 2022
This was referenced Jul 11, 2022
Merged
7 tasks
Very interested in making Gazebo simulate the added mass better. Any extra links I can read up that is not mention here? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
enhancement
New feature or request
MBARI buoy
Sponsored by MBARI buoy sim project: https://github.com/osrf/buoy_sim
physics
Involves Ignition Physics
Desired behavior
Gazebo's Hydrodynamics plugin can be used to apply forces due to added mass (i.e. the forces required to dislocate the fluid around a body moving underwater). However, using a plugin to apply such forces causes the simulation to be less stable, because it's reacting to the acceleration results at the end of a time step. Ideally, the added mass effect would be accounted for as part of the body's 6x6 inertia matrix.
Alternatives considered
Keep using the approach on the
Hydrodynamics
plugin.Implementation suggestion
DART's Inertia class can be initialized with any arbitrary 6x6 spatial inertia tensor. So we could expose that to Gazebo users by:
Inertial
. It would be interesting to hold the mass matrix separately from the current mass matrix so they can be inspected and manipulated separately. Add fluid added mass to inertial gz-math#459Other nice-to-haves:
Additional context
The text was updated successfully, but these errors were encountered: