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

Bugfix: if "false" is always True (backport #617) #640

Merged
merged 1 commit into from
Nov 8, 2024

Commits on Nov 8, 2024

  1. Bugfix: if "false" is always True (#617)

    There is an issue in this launch file when passing the string 'false' as
    an argument. In Python, non-empty strings are always evaluated as True,
    regardless of their content. This means that even if you pass 'false',
    the system will still evaluate it as True.
    
    This bug results in the launch system incorrectly calling the OnShutdown
    method twice. When any ROS launch action invokes a RosAdapter, it
    triggers the following exception: "Cannot shutdown a ROS adapter that is
    not running."
    
    To temporarily work around this issue, you can launch gz_sim_launch.py
    with the on_exit_shutdown argument set to an empty string. This prevents
    the erroneous shutdown sequence and avoids the associated exception.
    
    Signed-off-by: Ignacio Vizzo <[email protected]>
    (cherry picked from commit 1e30af0)
    nachovizzo authored and mergify[bot] committed Nov 8, 2024
    Configuration menu
    Copy the full SHA
    4a82600 View commit details
    Browse the repository at this point in the history