open-rmf / free_fleet

A free fleet management system.
Apache License 2.0
156 stars 65 forks source link

Fix for busy wait #120

Closed aaronchongth closed 1 year ago

aaronchongth commented 2 years ago

Using updated task request to signal that the client doesn't need to wait for remaining time anymore

Signed-off-by: Aaron Chong aaronchongth@gmail.com

Bug fix

Fixed bug

Fixes: https://github.com/open-rmf/free_fleet/issues/119

Fix applied

Fixed a possibly unintended busy wait for a long period in a while loop, instead of a periodic shorter wait.

Previous implementation performs a busy wait when arriving earlier at a waypoint, and blocks any updates to goals as well. This PR updates the task ids first before the goal paths, which will stop the client from waiting any longer.