Skip to content

Allowing gazebo_world_file arg to be passed in #148

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

Merged

Conversation

timmarkhuff
Copy link
Contributor

@timmarkhuff timmarkhuff commented Jan 11, 2024

Currently, when running ros2 launch blue_bringup bluerov2.launch.py, the gazebo_world_file argument is not supported.

This PR allows users to pass in the argument. If no argument is provided, bluerov2_underwater.world will be used.

@evan-palmer evan-palmer self-requested a review January 11, 2024 20:47
Copy link
Collaborator

@evan-palmer evan-palmer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Thank you for adding this. Would you be willing to update the other launch files as well to ensure consistency across the files?

@timmarkhuff
Copy link
Contributor Author

Looks good! Thank you for adding this. Would you be willing to update the other launch files as well to ensure consistency across the files?

This is done.

@evan-palmer evan-palmer self-requested a review January 12, 2024 07:17
evan-palmer
evan-palmer previously approved these changes Jan 12, 2024
@evan-palmer evan-palmer merged commit 01b385d into Robotic-Decision-Making-Lab:main Jan 13, 2024
@evan-palmer evan-palmer added backport-iron backport-humble Backport this PR to humble labels Jan 13, 2024
@evan-palmer
Copy link
Collaborator

Thanks for submitting this @timmarkhuff I will get it backported to Iron and Humble

mergify bot pushed a commit that referenced this pull request Jan 13, 2024
* Allowing gazebo_world_file arg to be passed in

* updating gazebo_world_file arg in launch file

* updating gazebo_world_file arg in launch file

* Pre-commit

---------

Co-authored-by: Evan Palmer <evanp922@gmail.com>
(cherry picked from commit 01b385d)
mergify bot pushed a commit that referenced this pull request Jan 13, 2024
* Allowing gazebo_world_file arg to be passed in

* updating gazebo_world_file arg in launch file

* updating gazebo_world_file arg in launch file

* Pre-commit

---------

Co-authored-by: Evan Palmer <evanp922@gmail.com>
(cherry picked from commit 01b385d)
evan-palmer pushed a commit that referenced this pull request Jan 14, 2024
* Allowing gazebo_world_file arg to be passed in

* updating gazebo_world_file arg in launch file

* updating gazebo_world_file arg in launch file

* Pre-commit

---------

Co-authored-by: Evan Palmer <evanp922@gmail.com>
(cherry picked from commit 01b385d)

Co-authored-by: Tim Huff <89954856+timmarkhuff@users.noreply.github.com>
evan-palmer pushed a commit that referenced this pull request Jan 14, 2024
* Allowing gazebo_world_file arg to be passed in

* updating gazebo_world_file arg in launch file

* updating gazebo_world_file arg in launch file

* Pre-commit

---------

Co-authored-by: Evan Palmer <evanp922@gmail.com>
(cherry picked from commit 01b385d)

Co-authored-by: Tim Huff <89954856+timmarkhuff@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-humble Backport this PR to humble
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants