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

Increase autopilot bank angle #2154

Merged
merged 4 commits into from Dec 1, 2020
Merged

Increase autopilot bank angle #2154

merged 4 commits into from Dec 1, 2020

Conversation

wpine215
Copy link
Member

Together with #2140, should improve autopilot performance

Summary of Changes

Increase autopilot max bank angle from 20 to 25

Discord username (if different from GitHub): Iceman#5540

How to download the PR for QA

Every new commit to this PR will cause a new A32NX artifact to be created and uploaded.
The build script will have already been run with the latest changes, so no need to rerun it once you download the zip.

  1. Make sure you are signed in to GitHub
  2. Click on the Checks tab on the PR
  3. On the right side, click on the Artifacts drop down and click the A32NX link

@wpine215 wpine215 added this to the v0.5.0 milestone Nov 30, 2020
@Lazarus320
Copy link

Lazarus320 commented Dec 1, 2020

Category: QA Tester
Name: Lazarus#4286
Date of testing: Current date 1/12/2020
Version of the sim: 1.11.6
PR Tested: #2154
Object of test: Autopilot bank angle

Testing technique: Loaded c/d and did a short flight
Steps you've made to test the Pull Request:

  1. Took off normally
  2. Tested ap bank angles with various configurations
  3. Average bank
    ezgif com-gif-maker (3)
  4. Maximum bank achieved
    ezgif com-gif-maker (4)

Flight notes (If required): The bank angle varies from 25 to approximately 29 which is according to what I found online from other pilots is fine since the ap bank angle will always differ depending on speed, mode, etc

Issues (If Any): N/A
Overall Rating: Good
Conclusions of the testing: The bank never exceeded 33 degrees so all looks good to me.

EDIT: Forgot to mention that the ap turn rate is more stable and better than before.

@wpine215 wpine215 merged commit 1297f87 into master Dec 1, 2020
@wpine215 wpine215 deleted the increase-bank-angle branch December 1, 2020 04:42
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.

None yet

2 participants