ArduPilot / MissionPlanner

Mission Planner Ground Control Station for ArduPilot (c# .net)
http://ardupilot.org/planner/
GNU General Public License v3.0
1.75k stars 2.38k forks source link

Warn user that DF logs cannot be viewed, cleared, downloaded while vehicle is armed. #1880

Open Kelly-Foster opened 6 years ago

Kelly-Foster commented 6 years ago

Currently running MP v1.3.56.4 with ArduPlane V3.9.0-dev / ChibiOS firmware on Matek F405-Wing flight controller. FC has integrated SD card reader. DF logging generally works well, as does terrain following. However when user tries to view/clear/download DF logs while armed, MP attempts to do so, and comes back ~30 seconds later with an unfriendly traceback depicted below. Far better, IMHO, to warn user that DF logs cannot be accessed while vehicle is armed.

mp error on df log fetch while armed

meee1 commented 6 years ago

ill add an armed check

Kelly-Foster commented 6 years ago

Thanks Michael. Happy to test once ready.

Kelly

meee1 commented 6 years ago

at this point its just going to be a message box as you open the screen, asking you not to be armed.

Kelly-Foster commented 6 years ago

Thats fine, and definitely better than current behavior.