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

Add workaround for stuck-on-launch headless simulation #398

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

snoato
Copy link
Contributor

@snoato snoato commented Nov 15, 2020

This PR addresses robocup-logistics/rcll-refbox#87 by providing a functional workaround.

The headless simulation (tmux based, as used in cx-simtest.bash), makes use of rcll-refbox-instruct to start the game. Due to a bug in boost::asio::io_service, it might get stuck without instructing the refbox. The following PR monitors the execution, retries after 2secs and repeats this for up to 10 times.

The headless simulation (tmux based, as used in cx-simtest.bash), makes
use of rcll-refbox-instruct to start the game. Due to a bug in
boost::asio::io_service, it might get stuck without instructing the
refbox. The following workaround monitors the execution, retries after
2secs and repeats this for up to 10 times.
@snoato snoato added the bug Something isn't working label Nov 15, 2020
@morxa
Copy link
Member

morxa commented Nov 15, 2020

At first glance, this looks like a hack. What's the actual bug in boost asio?

@snoato
Copy link
Contributor Author

snoato commented Nov 15, 2020

At first glance, this looks like a hack. What's the actual bug in boost asio?

Yes I agree, it's definitely not a solution, just a workaround I've been using since August. Victor and I spent the good part of an hour trying to find what the problem is (described in the linked issue), but it is somewhere within boost.

@morxa
Copy link
Member

morxa commented Nov 15, 2020

Maybe this helps?
https://stackoverflow.com/a/33575205/2785041

@snoato
Copy link
Contributor Author

snoato commented Nov 15, 2020

Maybe this helps?
https://stackoverflow.com/a/33575205/2785041

Looks somewhat promising, I'll look into it! Thank you!

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
None yet
Development

Successfully merging this pull request may close these issues.

2 participants