locusrobotics / aiorospy

asyncio wrapper for rospy
MIT License
40 stars 8 forks source link

Add 5 second delay when retrying service after exception #40

Closed abencz closed 2 years ago

abencz commented 2 years ago

Seems like aiorospy should wait a little before trying again, otherwise it ends up spamming the affected service. In particular, a service responding with ServiceException may be "normal" and there's no reason to think that calling the service again right away will succeed.

I'm not sure if this is the right solution though or if something more complicated (e.g. a timeout parameters somewhere) is needed.