Skip to content

Plane: tiltrotor: vectoring: correct roll sign and always apply throttle scale - #34086

Open
IamPete1 wants to merge 1 commit into
ArduPilot:masterfrom
IamPete1:fixTiltrotorVectoringRoll
Open

Plane: tiltrotor: vectoring: correct roll sign and always apply throttle scale#34086
IamPete1 wants to merge 1 commit into
ArduPilot:masterfrom
IamPete1:fixTiltrotorVectoringRoll

Conversation

@IamPete1

Copy link
Copy Markdown
Member

Summary

fixes #34041 It seems that vectoring for roll is backwards for tiltrotors. This only has significant effect at high tilt angles (but not at max tilt where there is no room left for vectoring).

Think of the front left motor in vtol flight. It would tilt forwards for CW (positive) yaw. If it were tilted forward then it should tilt up for right (positive) roll. Forward for yaw and up for roll are opposite so the sign in the existing calculation is incorrect.

This also applies the throttle scale factor to both roll and yaw, previously it was applied to yaw only.

Classification & Testing (check all that apply and add your own)

  • Checked by a human programmer
  • Non-functional change
  • No-binary change
  • Infrastructure change (e.g. unit tests, helper scripts)
  • Automated test(s) verify changes (e.g. unit test, autotest)
  • Tested manually, description below (e.g. SITL)
  • Tested on hardware
  • Logs attached
  • Logs available on request

Description

I attempted to come up with a auto test with this and failed. This really needs testing in realflight or IRL.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

QPLANE Tiltrotor Roll Control is Reversed During TRANSITION_AIRSPEED_WAIT (Sign Error)

1 participant