search

BrainFPV / TauLabs

http://www.brainfpv.com
Other
27 stars 9 forks source link

More detailed alarms. #36

Closed dustin closed 8 years ago

dustin commented 8 years ago

This is based on the work I did in MSP in TL proper. It's a slightly more condensed form to store the names of alarms, but should fail to compile if new alarms are added and not described here, provides more detail on system configuration and manual control alarms, and considers the boot cause to be an alarm for a bit at startup so you can see why it booted.

Review on Reviewable

dustin commented 8 years ago

I've not yet actually tested this, but it's probably really close to right.

fujin commented 8 years ago

Tested OK on bench with power, no props. Saw detailed alarms in HUD. amazing work. 10/10 :100: :+1:

fujin commented 8 years ago

Tested OK during autotune session, resulting autotune flew well. No adverse side effects, saw variety of CONFIG:UNSAFE, etc after autotune. Very powerful stuff.

dustin commented 8 years ago

Going to close this -- we've got a superset of the functionality in dRonin.

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io