Skip to content

Plane: Fix pitch stick mixing full override #29841

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rubenp02
Copy link
Contributor

Fixed a bug where, in certain cases, the pitch setpoint couldn't be fully overridden via stick mixing as intended. The issue occurred because the pitch offset applied at full stick deflection was calculated as twice the maximum pitch angle in the nudge direction, and since pitch limits are often asymmetric, this could be insufficient.

For example, with the default pitch range of [-25, 20], if the commanded pitch is -25º and the user nudges it fully upwards, the maximum resulting pitch would be -25 + 20 * 2 = 15º, falling short of the intended 20º.

Additionally, the non-linear stick input mapping has been replaced with a linear one, which is more suitable for pitch control.

Fixed a bug where, in certain cases, the pitch setpoint couldn't be
fully overridden via stick mixing as intended. The issue occurred
because the pitch offset applied at full stick deflection was calculated
as twice the maximum pitch angle in the nudge direction, and since pitch
limits are often asymmetric, this could be insufficient.

For example, with the default pitch range of [-25, 20], if the commanded
pitch is -25º and the user nudges it fully upwards, the maximum
resulting pitch would be -25 + 20 * 2 = 15º, falling short of the
intended 20º.

Additionally, the non-linear stick input mapping has been replaced with
a linear one, which is more suitable for pitch control.
@IamPete1 IamPete1 added the Plane label Apr 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants