search

equinor / flotilla

Flotilla is the main point of access for operators to interact with multiple robots in a facility.
Eclipse Public License 2.0
16 stars 38 forks source link

Fix confirm pressure/battery dialogs #1372

Closed Eddasol closed 9 months ago

Eddasol commented 9 months ago

Assumes bar as unit for all pressure values. Is dependent on https://github.com/equinor/flotilla/pull/1343

Eddasol commented 9 months ago

If pressure is too low: image

If battery is too low: image

Currently these dialogs won't pop up if scheduling echo missions, but will pop up if insufficient values when scheduling through inspection plan or rerunning mission

mrica-equinor commented 9 months ago

LGTM We should add a dialog that informs the user if both battery and pressure level are out of range. Can be solved in a different PR.

Agree with adding a third dialog so the operator doesn't go out to pressurize the robot and then when comes back realizes that has to go out again to charge it as well. But agree that can be done in another PR