-
Notifications
You must be signed in to change notification settings - Fork 16.6k
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
PPM Summ Out Signal Failsave detection #374
Comments
djmad, |
Within my tests failsafe never activates. Maybe APM makes a hold if no signal arrives? |
i try to recover the logfiles |
@gregmann, |
The Point is, that in my case the PWM failsafe was at 900, which is now profed by graupner and futuba that they DONT deliver a failsafe signal, they only shut down the protocoll. I arduciópter 3.0.0rc6 |
djmad,
|
as the sumo on channel 3 in my case goes from ~ 907 - 1890 i maybe did a unlucky setting :-( |
I believe we determined that in this case the FS_THR_VALUE parameter was set to 900 which is why arducopter did not recognise the failsafe. I'll add a pre-arm check that these values are correct so I'm closing this issue. |
I want to report that the Failsafe handling in the case of a RC-Signal Loss with a Sum Signal connection (SUMO8) does not work correctly
Graupner and Futuba changed in the latest Updates of there receivers the failsafe output
the following lines are translated from the Graupner support
"after a request from Flybarless, DJI and Mikrocopter we shut down the Sumo Signal in a case of error"
the problem at arducopter is following:
If the sumo shuts off every channel is on PWM900 which if you are actually in acro mode generates a very fast uncontrollable spin.
now the RTL kicks in (some time later... to late) because the Channel 1 too is below 900 and tries to capture the copter again. at this moment i already lost around 30m of height, the copter flipped over multiple axis... REST IN PIECES
we need a failsafe which triggers on SUMO-signal loss and does not interpret the non existent inputs as PWM900 for all 8 channels, which is really really bad
The text was updated successfully, but these errors were encountered: