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

Create ros_gz_spawn_model.launch #604

Merged
merged 15 commits into from
Oct 28, 2024

Conversation

Amronos
Copy link
Contributor

@Amronos Amronos commented Sep 6, 2024

🎉 New feature

Summary

Created ros_gz_spawn_model.launch taking inspiration from files like ros_gz_sim.launch.
Made to work after #606 is merged.

Test it

Something like the following command can be used:
ros2 launch ros_gz_sim ros_gz_spawn_model.launch world:=empty file:=$(ros2 pkg prefix --share ros_gz_sim_demos)/models/vehicle/model.sdf entity_name:=my_vehicle x:=5.0 y:=5.0 z:=0.5 bridge_name:=spawn_model_bridge config_file:=/path/to/config_file

Checklist

  • Signed all commits for DCO
  • Added tests
  • Added example and/or tutorial
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

@ahcorde ahcorde requested a review from caguero September 9, 2024 11:39
Signed-off-by: Aarav Gupta <[email protected]>
@Amronos Amronos requested a review from caguero September 9, 2024 16:26
@caguero
Copy link
Contributor

caguero commented Sep 10, 2024

We'll have to wait to merge this patch right after the Gazebo Ionic release considering that isn't fixing a critical but.

@Amronos
Copy link
Contributor Author

Amronos commented Oct 25, 2024

Should be ready to merge now.

@ahcorde
Copy link
Collaborator

ahcorde commented Oct 25, 2024

https://github.com/Mergifyio backport jazzy

Copy link
Contributor

mergify bot commented Oct 25, 2024

backport jazzy

🟠 Waiting for conditions to match

  • merged [📌 backport requirement]

@Amronos
Copy link
Contributor Author

Amronos commented Oct 25, 2024

#622 Should also be backported to Jazzy.

@Amronos Amronos requested a review from ahcorde October 28, 2024 12:46
@ahcorde ahcorde merged commit 02faa5b into gazebosim:ros2 Oct 28, 2024
4 checks passed
@ahcorde
Copy link
Collaborator

ahcorde commented Oct 28, 2024

https://github.com/Mergifyio backport jazzy

Copy link
Contributor

mergify bot commented Oct 28, 2024

backport jazzy

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Oct 28, 2024
Signed-off-by: Aarav Gupta <[email protected]>
Co-authored-by: Alejandro Hernández Cordero <[email protected]>
(cherry picked from commit 02faa5b)
@Amronos Amronos deleted the xml-ros_gz_spawn_model branch October 28, 2024 14:42
ahcorde pushed a commit that referenced this pull request Oct 28, 2024
Signed-off-by: Aarav Gupta <[email protected]>
Co-authored-by: Alejandro Hernández Cordero <[email protected]>
(cherry picked from commit 02faa5b)

Co-authored-by: Aarav Gupta <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants