Skip to content
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

Add option to force disarmed values when CBRK_IO_SAFETY is set #12457

Closed
jlecoeur opened this issue Jul 10, 2019 · 1 comment · Fixed by #12806
Closed

Add option to force disarmed values when CBRK_IO_SAFETY is set #12457

jlecoeur opened this issue Jul 10, 2019 · 1 comment · Fixed by #12806

Comments

@jlecoeur
Copy link
Contributor

jlecoeur commented Jul 10, 2019

Describe problem solved by the proposed feature
It is sometimes desired to lock all outputs to the disarmed values when disarmed. This is not possible when the safety switch is disabled (by setting CBRK_IO_SAFETY). In this case, the system is either in prearmed or in armed mode, and the servos are always free to move.

Describe your preferred solution
Add an option to lock all outputs to the disarmed values when in prearmed mode. This could be a parameter named CBRK_IO_PREARMED:

  • CBRK_IO_PREARMED = 0: current behaviour
  • CBRK_IO_PREARMED = 42: all outputs locked to disarmed values when disarmed or prearmed.

Describe possible alternatives

  • force disarmed values when CBRK_IO_SAFETY is set and the system is not armed. This would be an easier solution, but might break someone's workflow.

Additional context
The current behaviour is based on the assumption that (i) rotor commands are scaled with thrust command, like it is the case for multirotors, and (ii) flap motion is harmless.
However this is not always true:

  • a rotor can be mixed to roll only. It is thus not stopped by a low thrust value, and starts spinning unexpectedly. This can be dangerous.
  • servos may drive heavy/large parts, which are better locked when disarmed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants