-
Notifications
You must be signed in to change notification settings - Fork 13
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
Ignition to Gazebo migration #237
Conversation
Signed-off-by: Marco A. Gutierrez <[email protected]>
Signed-off-by: Marco A. Gutierrez <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@marcoag I see a few places that still talk about ignition
:
./README.md
./docker/empty_world/Dockerfile
./docker/tests/Dockerfile
./docker/base/Dockerfile
./docker/debug_integration/Dockerfile
./lrauv_system_tests/src/LRAUVController.cc
./lrauv_system_tests/worlds/buoyant_tethys_at_depth.sdf
./lrauv_system_tests/worlds/buoyant_tethys.sdf
./lrauv_description/models/tethys_equipped/model.sdf
Signed-off-by: Marco A. Gutierrez <[email protected]>
Signed-off-by: Marco A. Gutierrez <[email protected]>
Signed-off-by: Marco A. Gutierrez <[email protected]>
Thanks for the check!
This one is left for the last update as shown in #234.
These will also come in later update as part of the mbari side update, see #234.
This one has a line here that might depend on docker changes, so I left it until docker changes are done.
Updated these and some extra ones I found on the way. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. We need to update the lrauv-application
side and the wiki now.
Signed-off-by: Marco A. Gutierrez <[email protected]>
Signed-off-by: Marco A. Gutierrez <[email protected]>
Signed-off-by: Marco A. Gutierrez <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CI is happy so I'm happy.
Co-authored-by: Michel Hidalgo <[email protected]>
Signed-off-by: Marco A. Gutierrez <[email protected]>
Work towards #234.
Modifications needed for the Ignition to Gazebo name changing migration.
Signed-off-by: Marco A. Gutierrez [email protected]