MissionPlanner icon indicating copy to clipboard operation
MissionPlanner copied to clipboard

HUD: add ESC health label

Open rmackay9 opened this issue 9 months ago • 1 comments

Similar to our EKF and Vibration status reporting, I think we should add an ESC health reporting label and pop-up window. The label should probably be white most of the time but turn red if the ESCs start reporting errors. @peterbarker says that the SYS_STATUS message already includes a MAV_SYS_STATUS_SENSOR_MOTOR_OUTPUTS bit that AP sets if the ESCs start producing errors.

Perhaps the label should only appear if the ground station is receiving ESC_TELEMETRY messages.

If the user clicks on this label we should probably display the RPM, Temperature and error count for the ESCs

This comes in response to this 4.6 user crash in which the ESCs started reporting errors a full 4 minutes before the vehicle actually crashed.

rmackay9 avatar May 29 '25 03:05 rmackay9

Similar to our EKF and Vibration status reporting, I think we should add an ESC health reporting label and pop-up window. The label should probably be white most of the time but turn red if the ESCs start reporting errors. @peterbarker says that the SYS_STATUS message already includes a MAV_SYS_STATUS_SENSOR_MOTOR_OUTPUTS bit that AP sets if the ESCs start producing errors.

Note quite what I said. AP currently unconditionally says "healthy" - but we could fix that.

[1:33 PM]Peter Barker: A sys_status bit already exists for that.  MAV_SYS_STATUS_SENSOR_MOTOR_OUTPUTS
[1:35 PM]Peter Barker: ... we currently unconditionally set the "health" bit corresponding to that enumeration value to true.  Should not be too difficult to ask some library or another whether it should be false.  Or possibly several libraries, all of which have to say "healthy" for it to be true.

peterbarker avatar May 29 '25 04:05 peterbarker