-
Notifications
You must be signed in to change notification settings - Fork 17.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Scripting: added copter deploy example #26651
Conversation
I guess we can't use the existing arm() binding because of one of these issues?
|
this one, plus I think it is clearer to have arm_force so users don't use it by mistake |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the binding name, "arm_force".
Ping @tridge - should be an easy fix/merge |
@@ -96,6 +96,7 @@ class AP_Arming { | |||
// these functions should not be used by Copter which holds the armed state in the motors library | |||
Required arming_required() const; | |||
virtual bool arm(AP_Arming::Method method, bool do_arming_checks=true); | |||
virtual bool arm_force(AP_Arming::Method method) { return arm(method, false); } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be preferable to check the number of arguments being passed to the C function (lua_gettop
?) rather than creating methods like this, IMO.
this monitors climb with raw pressure and force arms in LAND mode on descent
this script was written to help a student project dropping a copter from a rocket, but I have wanted force arm in other places