open-rmf / rmf_task

RMF library for managing task allocations
Apache License 2.0
22 stars 22 forks source link

More cautious sequence advancing #102

Closed mxgrey closed 9 months ago

mxgrey commented 10 months ago

There is a risk that a downstream event implementation could cause the sequence to advance multiple times, e.g. by incorrectly calling its finished callback. This PR prevents that by checking whether we have already processed the event advancement when the finished callback is triggered.