open-rmf / free_fleet

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

develop: Add field to RobotState to track command completion #114

Open chianfern opened 2 years ago

chianfern commented 2 years ago

New feature implementation

Implemented feature

Added command_completed, a boolean that indicates the completion of the current command.

Implementation description

Previously, if a command was completed, the RobotState's command_id should be a nullopt. This led to the issue where if a new command is issued and instantly completed, the following RobotState would have a null command_id and users of the free_fleet Manager would be unable to determine if the command was successful or was not received.

With the new command_completed field, the completion status is explicit and command_id always has the id of the latest command.