Scripted Arming Checks #30009
Scripted Arming Checks #30009
Conversation
Well I am hoping to simply extend it to do other vehicles, rather than having different scripts. I'm expecting there will be a lot of common checks. That's my theory anyway. |
|
Hi @timtuxworth, OK great, if it's meant to work on all vehicles, could you test this by running it on Copter (and perhaps Rover if you have time)? I suspect the calls to get the Q_ENABLE parameter will mean the script errors out on copter |
timtuxworth
force-pushed
the
pr-scripted-arming-checks
branch
from
49a9061 to
e12b18b
Compare
Done! And fixed because yes you were right there were a couple of errors. Now works with at least Plane, Copter, Rover. There are 4 checks common to all and 6 common between Plane and Copter. |
timtuxworth
force-pushed
the
pr-scripted-arming-checks
branch
from
e12b18b to
0d015c3
Compare
timtuxworth
force-pushed
the
pr-scripted-arming-checks
branch
from
0d015c3 to
28a31d0
Compare
| self.wait_text("Cleared: FOLL_SYSID not set", check_context=True) | ||
|
|
||
| ''' Need a healthy camera mount defined for this to work ''' | ||
| self .start_subsubtest("ArmCk: Mount SYSID must be match FOLL_SYSID") |
There was a problem hiding this comment.
| self .start_subsubtest("ArmCk: Mount SYSID must be match FOLL_SYSID") | |
| self.start_subsubtest("ArmCk: Mount SYSID must be match FOLL_SYSID") |
Extra space?
|
Very nice Tim! |
timtuxworth
force-pushed
the
pr-scripted-arming-checks
branch
from
28a31d0 to
8307ecf
Compare
This adds a Lua Applet for scripted arming checks. Each check can be set as mandatory (generates a prearm fail), warning (displays a message, but allows arming) or disabled by setting a parameter.
There is a set of standard checks that I find useful, of course new ones can be added. Any suggestions appreciated.
This is designed to get around the controversy that often arises about adding arming checks to c++. To some people particular checks are essential, while to others they are unnecessary or annoying.
This works for plane, copter and rover for now, but as a base could easily be extended to other vehicles.