Progress Bar NaN Issue

[amalnanavati]

When the robot transitions to a RobotMotion state, it first momentarily renders the progressBar with NaN before actually rendering the progress. See this screen recording.

This issue is about understanding why that is happening and fixing it.

[Raidakarim]

I don't see this bug on my end. I checked in both Firefox and Chromium browser in Ubuntu VM. Not sure if it is your setting-specific(?!).

[amalnanavati]

I delved more into this bug. It seems like for some reason, the first feedback message that is received on a RobotMotion is sometimes a feedback message from the previous action call. This is reproduceable on my computer if I log the feedback message at the top of the callback function, and then alternate between the same few actions multiple times (e.g., MoveToRestingPosition, MoveToMouth).

This makes me think that either:

1. We are incorrectly using ros2actions from roslibjs.
2. There is a bug in ros2actions from roslibjs (e.g., they should have called removeAllEventListeners on destroyActionClient or something similar).
3. There is a bug in rosbridge_websocket.

The reason I know it's not a bug in ada_feeding is that when we call the same actions from the command line interface, we don't see this behavior.