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

ros_ign_gazebo- ign_args launch arg not respected in ign_gazebo.launch.py #308

Closed
mwcondino opened this issue Oct 7, 2022 · 4 comments
Closed
Assignees
Labels
bug Something isn't working

Comments

@mwcondino
Copy link

mwcondino commented Oct 7, 2022

Environment

  • OS Version: Ubuntu 22.04
  • Source or binary build?
    ros-humble-ros-ign-gazebo at 0.244.6-1jammy.20220916.172640
    libignition-gazebo6:amd64 at 6.12.0-1~jammy

Description

It looks like the ign_gazebo.launch.py launch file may not respect ign_args in a humble environment with Ignition Fortress installed.

Steps to reproduce

  1. Ensure ros-humble-ros-ign-gazebo is installed
  2. Try running a gazebo session headless through the launch file:
ros2 launch ros_ign_gazebo ign_gazebo.launch.py ign_args:="-s"
  1. You should see a gui pop up after this - this indicates the ign_args parameter is not set.

Workaround

Using gz_args instead of ign_args appears to be working. In addition, I think if you force the ignition version like this....

ros2 launch ros_ign_gazebo ign_gazebo.launch.py ign_args:="-s" ign_version:=6

the bug is no longer evident.

It's probably best to update launch files to use gz_args as a fix here, since that is more future facing and ign_* args are deprecated.

@mwcondino mwcondino added the bug Something isn't working label Oct 7, 2022
@osrf-triage osrf-triage added this to Inbox in Core development Oct 7, 2022
@mjcarroll mjcarroll self-assigned this Oct 11, 2022
@mjcarroll mjcarroll moved this from Inbox to In progress in Core development Oct 11, 2022
@mjcarroll
Copy link
Collaborator

@mwcondino please look at #309 and see if this resolves the issue.

@mjcarroll
Copy link
Collaborator

It's probably best to update launch files to use gz_args as a fix here, since that is more future facing and ign_* args are deprecated.

This is ideal long-term, but there should be some middle ground with deprecation warnings, that is what I have implemented.

@mjcarroll
Copy link
Collaborator

#309 has been released to humble and will be available in the next sync. If this doesn't resolve it, please feel free to reopen.

Core development automation moved this from In progress to Done Oct 14, 2022
@mwcondino
Copy link
Author

Thanks @mjcarroll, that fix looks good to me!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

No branches or pull requests

2 participants