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.

Sign up for GitHub

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

Jump to bottom

Update LRAUV model upstream to use new hydrodynamic parameters. #2575

Open
arjo129 opened this issue Sep 2, 2024 · 0 comments · May be fixed by #2608
Open

Update LRAUV model upstream to use new hydrodynamic parameters. #2575

arjo129 opened this issue Sep 2, 2024 · 0 comments · May be fixed by #2608
Assignees
@arjo129
Labels
boost the docs Documentation deployment, usability and parity with classic

Comments

@arjo129
Copy link
Contributor

arjo129 commented Sep 2, 2024 •
edited
Loading

With the deprecation of added mass using the hydrodynamics plugin, we need to update the upstream lrauv model on fuel for the demo world.
@arjo129 arjo129 added the bug Something isn't working label Sep 2, 2024
@arjo129 arjo129 self-assigned this Sep 2, 2024
@arjo129 arjo129 mentioned this issue Sep 2, 2024
Closed
@arjo129 arjo129 added boost the docs Documentation deployment, usability and parity with classic and removed bug Something isn't working labels Sep 2, 2024
@arjo129 arjo129 changed the title Update LRAUV model upstream Update LRAUV model upstream to use new hydrodynamic parameters. Sep 2, 2024
@arjo129 arjo129 mentioned this issue Sep 3, 2024
Closed
arjo129 added a commit that referenced this issue Sep 9, 2024
@arjo129
Update added-mass of lrauv model and remove added mass definitions from
daaf518 
examples

Fixes #2575

Since, we have decided to deprecate added mass in the hydrodynamics
plugin in favour of Rigid Bodies, we should update our
examples to avoid the warning.

TODO: Update maritime tutorials and migration guide.

Signed-off-by: Arjo Chakravarty <arjoc@intrinsic.ai>
@arjo129 arjo129 linked a pull request Sep 9, 2024 that will close this issue
Open
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@arjo129 arjo129

Labels
boost the docs Documentation deployment, usability and parity with classic
Projects
Core development
Status: Inbox
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update added-mass of lrauv model and remove added mass definitions from examples gazebosim/gz-sim
1 participant
@arjo129