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

Potential Bug of wait_for_service method Related To Ros Service-Client Infrastructure #15

Open
CihatAltiparmak opened this issue Jul 23, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@CihatAltiparmak
Copy link
Member

CihatAltiparmak commented Jul 23, 2024

https://github.com/CihatAltiparmak/moveit_middleware_benchmark/blob/b91a2434a10033d51122c52ec0685392d41c809f/src/scenarios/scenario_basic_service_client.cpp#L49-L62

As you can see in above code part, we must wait little bit more once wait_for_service returns True which means service is ready and good to go. Because if you send a request to server once wait_for_service returns true, server sometimes may not send a response to client and this gives rise to get stuck in waiting response from server. Our benchmark implementation seems to run well. Just for noting and investigating

@CihatAltiparmak CihatAltiparmak added the bug Something isn't working label Jul 23, 2024
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

No branches or pull requests

1 participant