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
Plane: Quadpalne: move tiltrotor functionality to own class #18579
Conversation
d81da5e
to
e1ce903
Compare
All seems to work as expected with the Griffin in RF |
e1ce903
to
921b352
Compare
rebased on mode changes and re-tested param conversion. |
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.
queue this after peters quadplane enable PR
2c29127
to
e447308
Compare
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.
@IamPete1 testing fails badly on Griffin model in RF9. The motors end up tilting down:
it tries to take off like that and does a nose down
parameters attached
griffin.txt
e447308
to
d1c7797
Compare
d1c7797
to
8e3b5a8
Compare
I had "fixed" all of those I have done it properly now. |
Makes quadplane slightly cleaner and lets us hide tilt params. Removes remaining confusion between tailsitters and tiltrotors.