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

robotinterface: Close as soon as one external device is closing #619

Merged
merged 1 commit into from
Apr 4, 2022

Conversation

traversaro
Copy link
Member

@traversaro traversaro commented Mar 31, 2022

This PR implements the idea described in #582 (comment) .

This fixes #582 .

This requires #618 to be merged before, to preview just the content of this PR before that PR is merged, check the commit 4d5ff7e .

@traversaro
Copy link
Member Author

Testing on an internal model (stickBot) that was affected by this problem, with this modification we switched from the model crashing to hanging with this output:

[INFO] interrupt1 phase starting...
[INFO] interrupt1 phase finished.
[INFO] shutdown phase starting...
[INFO] Entering action level 2 of phase shutdown
[INFO] Executing detach action, level 2 on device wholebodydynamics with parameters []

I need to investigate more, but I suspect this is actually some problem of the YARP devices launched, not of the gazebo_yarp_robotinterface plugin or this modifications.

@traversaro
Copy link
Member Author

Testing on an internal model (stickBot) that was affected by this problem, with this modification we switched from the model crashing to hanging with this output:

[INFO] interrupt1 phase starting...
[INFO] interrupt1 phase finished.
[INFO] shutdown phase starting...
[INFO] Entering action level 2 of phase shutdown
[INFO] Executing detach action, level 2 on device wholebodydynamics with parameters []

I need to investigate more, but I suspect this is actually some problem of the YARP devices launched, not of the gazebo_yarp_robotinterface plugin or this modifications.

Indeed, this was due to some bugs in #618 and in wholebodydynamics, once that has been fixed it was possible to respawn easily models that used gazebo_yarp_robotinterface:

stickbot_remove.mp4

CHANGELOG.md Outdated Show resolved Hide resolved
@traversaro
Copy link
Member Author

Thanks @Nicogene for the review. Now the history of the PR is clean.

@traversaro traversaro merged commit 992c1ab into master Apr 4, 2022
@traversaro traversaro deleted the fix582 branch April 4, 2022 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

A gazebo model which uses yarprobotinterface plugin crashes when the model is removed
2 participants