-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
VTOL Quad - replace VT_TRANS_THR #702
Conversation
By the way, can we delete this in tailsitter? This function also seems to have no effect(_thrust_transition always 0.0): |
Yes, you are absolutely right. We now have |
Thanks @RomanBapst - can you make the changes directly in this PR by editing the page? I'd prefer that :-) Otherwise I can do it and you review, but that's me guessing how this works, while since you "know" should be a few minutes change. |
@hamishwillee Hang on, I wasn't done yet I need to add some more info. I will collect it here and can also do the changes :-) |
Backtransition throttle VT_B_TRANS_THR:
Front transition throttle VT_F_TRANS_THR:
|
Signed-off-by: RomanBapst <[email protected]>
Signed-off-by: RomanBapst <[email protected]>
@hamishwillee Aight, please have a look |
Looks great. Thanks very much! Merging |
PS Re #702 (comment) That is pretty interesting but not relevant to THIS doc which only applies to standard vtol (quad). You have comments about this in the param itself, so the info is captured. |
Hi @RomanBapst (&/or @sanderux )
Link checker shows that
VT_TRANS_THR
disappeared at some point. Possibly replaced byVT_F_TRANS_THR
I've marked up the bits that are affected in the tuning guide. Can you please advise how this should "properly" change.
There is a degree of urgency because we'll be cutting the PX4 v1.11 release soon, and it would be nice if the docs were accurate-ish at this point!