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

VTOL Quad - replace VT_TRANS_THR #702

Merged
merged 4 commits into from
Mar 31, 2020
Merged

Conversation

hamishwillee
Copy link
Collaborator

Hi @RomanBapst (&/or @sanderux )

Link checker shows that VT_TRANS_THR disappeared at some point. Possibly replaced by VT_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!

@xdwgood
Copy link

xdwgood commented Mar 31, 2020

By the way, can we delete this in tailsitter?

This function also seems to have no effect(_thrust_transition always 0.0):
https://github.com/PX4/Firmware/blob/master/src/modules/vtol_att_control/tailsitter.cpp#L270

@RomanBapst
Copy link
Contributor

@hamishwillee

Possibly replaced by VT_F_TRANS_THR

Yes, you are absolutely right. We now have VT_F_TRANS_THR and VT_B_TRANS_THR for front-and backtransition respectively.

@hamishwillee
Copy link
Collaborator Author

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.

@RomanBapst
Copy link
Contributor

@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 :-)

@RomanBapst
Copy link
Contributor

Backtransition throttle VT_B_TRANS_THR:

  • only used for standard vtol (you can set negative values and then you can have reverse thrust if your ESC supports that)
  • tailsitters and tiltrotors use the throttle from the mc position controller

Front transition throttle VT_F_TRANS_THR:

  • only used for standard VTOL, pusher throttle will be ramped to that value

@RomanBapst
Copy link
Contributor

@hamishwillee Aight, please have a look

@hamishwillee
Copy link
Collaborator Author

Looks great. Thanks very much! Merging

@hamishwillee hamishwillee merged commit 6db5329 into master Mar 31, 2020
@hamishwillee
Copy link
Collaborator Author

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.
There might be more work to be done in https://docs.px4.io/master/en/config_vtol/vtol_back_transition_tuning.html - a quick scan would be appreciated to confirm yay/nay :-)

@hamishwillee hamishwillee deleted the hamishwillee-VT_TRANS_THR branch July 14, 2021 01:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants