Turtle/Flip after crash: Add options to use all the motors #5600
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
brycedjohnson
changed the title
mikeller
reviewed
Apr 1, 2018
src/main/flight/mixer.c
Outdated
|
|
||
|
|
||
| if (motorOutput < 0 && mixerConfig()->crashflip_motor_percent > 0) | ||
| { |
There was a problem hiding this comment.
Brace on the same line as the condition please, only functions use braces on a new line (K & R style).
src/main/flight/mixer.c
Outdated
|
|
||
| if (motorOutput < 0 && mixerConfig()->crashflip_motor_percent > 0) | ||
| { | ||
| motorOutput = -motorOutput * (float)mixerConfig()->crashflip_motor_percent/100.0f; |
There was a problem hiding this comment.
Spaces around operators please.
brycedjohnson
force-pushed
the
AllMotorTurtle2
branch
from
a344ff7
to
3e4e958
Compare
|
Got a bug with this right now, going to fix it... |
brycedjohnson
force-pushed
the
AllMotorTurtle2
branch
from
3e4e958
to
503a44a
Compare
|
It is fixed now. |
mikeller
previously approved these changes
Apr 4, 2018
|
Rebase after #5627 has been merged - that should fix the overflow. |
brycedjohnson
force-pushed
the
AllMotorTurtle2
branch
from
503a44a
to
9ca0b77
Compare
mikeller
approved these changes
Apr 6, 2018
ethomas997
added a commit
to ethomas997/betaflight
that referenced
this pull request
Apr 10, 2018
From: Turtle/Flip after crash: Add options to use all the motors betaflight#5600 betaflight#5600
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added crashflip_motor_percent. This will use the other motors while doing a turtle. This is typically only useful on lower power ducted quads. Setting to 0 will use just 2 motors which is the default. 50-70% seemed to work for getting a brushless tinywhoop flipped over easier.
Quick test here: Helps a lot!
http://www.youtube.com/watch?v=Qwh1MBNnZj4
Tinywhoop.com helped me with some motors and frames to test!