-
Notifications
You must be signed in to change notification settings - Fork 13.5k
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
Tailsitter mission problem #9300
Labels
Comments
Let's get a simple tailsitter mission added to CI after this is fixed.
…On Fri, Apr 13, 2018, 06:14 Roman Bapst ***@***.***> wrote:
@dagar <https:/dagar> FYI
Seems like tailsitter mission and position control is broken on master (it
does not track at all, roll setpoints changes sign frequently).
I found a working version not too long ago and will do a bisect.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#9300>, or mute the thread
<https:/notifications/unsubscribe-auth/AAFK6EdgfoxWmYm0ZN98eA-TdnxVlpyzks5toHqDgaJpZM4TTMxO>
.
|
@RomanBapst I won't have time to look into this today, but I have a hunch this could be the problem. https:/PX4/Firmware/blob/master/src/modules/fw_pos_control_l1/FixedwingPositionControl.cpp#L706-L718 If that's the case we really should centralize the tailsitter frame conversion directly in the attitude update method rather than having them spread out and done as needed. |
WIP PR to add tailsitters to CI mission testing. #9307 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
@dagar FYI
Seems like tailsitter mission and position control is broken on master (it does not track at all, roll setpoints changes sign frequently).
I found a working version not too long ago and will do a bisect.
The text was updated successfully, but these errors were encountered: