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

Lateral autopilot fix #2140

Merged
merged 1 commit into from Nov 30, 2020
Merged

Lateral autopilot fix #2140

merged 1 commit into from Nov 30, 2020

Conversation

wpine215
Copy link
Member

Fixes #2089

Summary of Changes

Fixes lateral autopilot/flight director deviation that arose since the last MSFS patch. Should fix wrongly accumulated integral term in the heading PID controller.

May need multiple cycles of extensive testing and iteration for tuning.

Screenshots (if necessary)

N/A

References

Special thanks to Sebastian from Asobo for helping to diagnose the issue and come up with this temporary solution.

Additional context

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 Autoflight ata-22 Do Not Merge Extensive Testing Needed More testing needed in this PR labels Nov 30, 2020
@wpine215 wpine215 added this to the v0.5.0 milestone Nov 30, 2020
@Snapmatics
Copy link
Contributor

SEBASTIAN THE KING

@wpine215
Copy link
Member Author

Tested it, full flight from cold-and-dark using SID's and STAR's. Engaged autopilot shortly after takeoff and kept it on until 1,000 feet RA before landing. Autopilot works with managed heading, selected heading, and capturing/tracking localizer, all without issue. The plane banks to its full range of motion (currently configured to 20 degrees) when necessary, both left and right. Plane follows flight plan without issue.

@armankir
Copy link
Contributor

Everyone says it works so I guess it works lol, will test and write a report soon.

@Snapmatics
Copy link
Contributor

Works for me

@FernandoPusineriWebMasterDesigner

Hello !how can I do to join this patch to the latest version of the mod ?, because I downloaded it from the code and when I start the sim it does not grab the mod, only the default version. Thank you !

@CaptainR3X
Copy link

CaptainR3X commented Nov 30, 2020

Category: QA Tester
Name: GustavRex #4347
Date of testing: Current date 30/11/2020
Version of the sim: 1.11.6.0
PR Tested: #2140
Object of test: Lateral AP fix

Testing technique: Flight; EKCH-ESGG
Steps you've made to test the Pull Request:

  1. Normal flight programming, P/S, takeoff
  2. Vectors on takeoff, climb via SID afterwards
  3. Cruise, arrival planning, descent
  4. Descent via SID, vectors to ILS intercept
  5. Landing

Flight notes (If required): Online, VatSim

Issues (If Any): None
Overall Rating: Good
Conclusions of the testing: AP followed plan, following issues were no longer present;

  • Heavy banking, fixed
  • Ap not following plan (managed hdg), fixed
  • HDG mode not working, fixed
  • ILS intercept - AP not aligning, fixed

Thanks for your hard work guys. You've outdone yourselves here <3

@BritishAvgeek
Copy link

Category: QA Tester
Name: BritishAvgeek#1436 (used to be Carter633 now changed to match discord)
Date of testing: Current date 30/11/2020
Version of the sim: 1.11.6.0
PR Tested: #2140
Object of test: Dynamic fix for lateral autopilot issues

Testing technique: short full flight comparable to a route flown the other day with the issues

Steps you've made to test the Pull Request:

  1. Spawned Cold&Dark EGCC, input all relevant data into aircraft
  2. Depart EGCC for EGLL flying the SANB1R SID - via HON - into the BNN4A arrival and cruise FL190
  3. Monitored systems for issues/track deviation
  4. Land and complete flight, shut down

Issues (If Any): None
Overall Rating: Good
Conclusions of the testing: Nice work!

@Benjozork
Copy link
Member

Hello !how can I do to join this patch to the latest version of the mod ?, because I downloaded it from the code and when I start the sim it does not grab the mod, only the default version. Thank you !

It will be in the development version within a few minutes

@LoungeFlyZ
Copy link
Contributor

@wpine215 can you describe what these config values change? just interested.

@ImenesFBW
Copy link
Contributor

Made two flights with problematic routes that haven't worked for me after the update.

Works great for me, no issues, thanks!

@dadavid77
Copy link

Thx guys, AP is working again !

@Benjozork Benjozork merged commit 4ded975 into master Nov 30, 2020
@FinalLightNL
Copy link
Contributor

Thankyou so much Sebastian !

@wpine215 wpine215 deleted the lateral-ap-patch branch November 30, 2020 21:39
lukecologne added a commit that referenced this pull request Nov 30, 2020
Co-authored-by: AsoboStudio <achat@asobostudio.com>
@TheoBearman
Copy link

TheoBearman commented Nov 30, 2020

Hate to rain on the parade but I think some fine-tuning is still required. Coupled with the 20-degree maximum bank it does take a while to get round corners as well as settle on a course, even in flat calm weather.

Perhaps an updated stable build which incorporates this fix would be useful while everything else in the dev version is smoothed out?

Thanks a lot for the efforts @wpine215 and @AsoboStudio.

Capture

@felixalmont09
Copy link

I was doing LFPG-LIRF just now and AP had the same left bank tendency on departure, HD wasnt working either. It was working in the dev version prior to the last one updated like 2h ago.

@Benjozork
Copy link
Member

This is a temporary fix to make the plane usable.

@Benjozork
Copy link
Member

I was doing LFPG-LIRF just now and AP had the same left bank tendency on departure, HD wasnt working either. It was working in the dev version prior to the last one updated like 2h ago.

This sounds like an issue with deadzones. Try setting them to 10%.

@felixalmont09
Copy link

felixalmont09 commented Nov 30, 2020 via email

@DeadManDrift
Copy link

Thank you so much for what you do!

I have noticed after updating to the current Dev build, my v speeds are weird. V1 shows -5 VR shows -4 and V2 is blank. If I fill in number based off of the performance calculator, I get an error that says V1/VR/V2 Disagree... Has anyone else noticed that? Usually my v speeds are blank and I do not get the error.

Thanks for your time!

@Benjozork
Copy link
Member

Thank you so much for what you do!

I have noticed after updating to the current Dev build, my v speeds are weird. V1 shows -5 VR shows -4 and V2 is blank. If I fill in number based off of the performance calculator, I get an error that says V1/VR/V2 Disagree... Has anyone else noticed that? Usually my v speeds are blank and I do not get the error.

Thanks for your time!

Hello, this has been fixed and is currently pending merge into the development version.

@DeadManDrift
Copy link

Thank you so much for what you do!
I have noticed after updating to the current Dev build, my v speeds are weird. V1 shows -5 VR shows -4 and V2 is blank. If I fill in number based off of the performance calculator, I get an error that says V1/VR/V2 Disagree... Has anyone else noticed that? Usually my v speeds are blank and I do not get the error.
Thanks for your time!

Hello, this has been fixed and is currently pending merge into the development version.

Thank you again!

@Awemeter
Copy link
Contributor

Awemeter commented Nov 30, 2020

Category: Trial QA tester
Name: Awemeter#7919
Date of testing: 11/30/2020
Version of the sim: 1.11.6.0
PR Tested:#2140
Object of test: Lateral autopilot fix

Testing technique: Went in sim and did a flight.
Steps you've made to test the Pull Request:

  1. Started C&D and planed mcdu normally at katl.
  2. Took off and engages autopilot. Autopilot follows its path and Flight director works properly.
  3. Landed at kmco

Flight notes (If required):

Issues (If Any): At cruise had very aggressive rocking.
Overall Rating: Good
Conclusions of the testing: Everything looks really good no problems found that are new to this or. The autopilot problem is fixed.

@wchafe
Copy link

wchafe commented Nov 30, 2020

Repeated flight that previously had issues with wing rocking initiated by setting manual hdg mode. Hdg mode did not cause wing rocking. Only issue was about 1 hour into the flight wing rocking started while nearing a waypoint. Rocking was moderate and did not change in amplitude. Turned off SEC2 momentarily, wing rocking stopped. Re-engaged SEC2, rocking do not return for a long time, but eventually happened again. Toggling SEC2 fixed it again.
Otherwise AP/FD appeared to work normally with one exception, which is probably an unrelated issue. That is, when manual HDG mode was engaged during CLM the Pitch mode annunciator on the ADI went blank. It returned when heading was changed back to managed mode. Repeated multiple times.

@felixalmont09
Copy link

So i updated the dev version again and AP is working now.

@fabianoheringer
Copy link

Updated, working now!

@npavia5
Copy link

npavia5 commented Dec 1, 2020

where is this link

@flybywiresim flybywiresim locked as off-topic and limited conversation to collaborators Dec 1, 2020
@Benjozork
Copy link
Member

Locked as conversation is going into irrelevant topics. Please seek installation assistance on our Discord and talk about the rocking issue in #2005.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Autoflight ata-22 Do Not Merge Extensive Testing Needed More testing needed in this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Autopilot systems not functioning after MSFS update