Skip to content

Switch to using ros_gz_sim to run Gazebo#344

Merged
luca-della-vedova merged 2 commits intomainfrom
luca/ros_gz_sim
Jan 13, 2026
Merged

Switch to using ros_gz_sim to run Gazebo#344
luca-della-vedova merged 2 commits intomainfrom
luca/ros_gz_sim

Conversation

@luca-della-vedova
Copy link
Member

Bug fix

Fixed bug

Brought up in open-rmf/rmf_visualization#88 (comment).
It seems argument substitution is behaving a bit oddly with our current way to run gazebo through the executable tag.
Specifically, on Rolling the gz_headless variable seems to break the launch command, as shown in the comment (it also doesn't work for me, but running in headless mode actually works).

Fix applied

I tried to change the content of the gz_headless variable to contain the whole command, change its position in the command, various combination of content but I can't quite figure out where the issue exactly lies. It's really very odd, sometimes only headless mode works, sometimes only normal mode works, but never both at the same time.
By contrast, using ros_gz_sim seems to always work. This seems to be also the officially recommended way to launch Gazebo based on the official docs.

GenAI Use

We follow OSRA's policy on GenAI tools

I tested this on both a Jazzy and a Rolling machine.

  • I used a GenAI tool in this PR.
  • I did not use GenAI

Signed-off-by: Luca Della Vedova <lucadv@intrinsic.ai>
Copy link
Member

@xiyuoh xiyuoh left a comment

Choose a reason for hiding this comment

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

Awesome! Tested this for both Jazzy and Rolling, am able to run demos for hotel and office. Thanks for the quick fix!

@luca-della-vedova
Copy link
Member Author

The CI failure is expected since the fixed rviz_common package is not part of the sync yet and not caused by this PR that doesn't touch anything related to Qt, will merge and bloom a new version

@luca-della-vedova luca-della-vedova merged commit 54d7bfa into main Jan 13, 2026
2 of 4 checks passed
@github-project-automation github-project-automation bot moved this from Inbox to Done in PMC Board Jan 13, 2026
@luca-della-vedova luca-della-vedova deleted the luca/ros_gz_sim branch January 13, 2026 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants