ardupilot icon indicating copy to clipboard operation
ardupilot copied to clipboard

AP_Parachute: Optional (CHUTE_OPTIONS:1) disarm before parachute release

Open python36 opened this issue 1 year ago • 2 comments

Based on discussion of PR #25570 and #25562.

Disarm before parachute release to avoid parachute tangling if parameter CHUTE_OPTIONS:1 = 0.

python36 avatar Feb 02 '24 10:02 python36

We have a option to enable safety switch on disarm, if that was set disarm would cause safety to come on which could in-turn inhibit your parachute release servo.

I'm not really sure how we can overcome that. It maybe such a edge case that is does not matter. Of course it only affects those methods that don't currently disarm.

IamPete1 avatar Feb 02 '24 23:02 IamPete1

Hi @IamPete1 . Please write a set of parameters and actions that will lead to this. I have tested many cases and cannot inhibit the parachute servo with the safety buttons.

python36 avatar Feb 14 '24 09:02 python36

if that was set disarm would cause safety to come on which could in-turn inhibit your parachute release servo.

@IamPete1 we currently always disarm on release, so this is adding an option to not do that. The exact order is likely not important as setting the PWM for a server isn't instantaneous anyway

tridge avatar Apr 06 '24 08:04 tridge

I think this is good, marked for dev call topic for discussion

tridge avatar Apr 06 '24 08:04 tridge

@Hwurzburg for docs, we need to note that BRD_SAFETY_MASK needs to be setup to allow chute servo when disarmed if you have the option to set safety on when you disarm

tridge avatar Apr 08 '24 23:04 tridge

We should fix the commit message.

rmackay9 avatar Apr 08 '24 23:04 rmackay9

I split the commits into subsystems and rebased

tridge avatar Apr 08 '24 23:04 tridge

Many thanks

python36 avatar Apr 12 '24 15:04 python36