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

refactor: Fadec rewrite using Cpp framework (2nd) #8622

Merged
merged 85 commits into from
Apr 15, 2024

Conversation

frankkopp
Copy link
Member

@frankkopp frankkopp commented Apr 12, 2024

Summary of Changes

Migration of the current fadec code for the A32NX and A380X to the cpp framework with cleanup and additional commenting for better understanding and maintainability.

No change in logic and also not further modularized into shared and specific parts as most of the A380X fadec code still uses the A32NX polynomials and tables. As this will surely change any further sharing of code should be done after when it is clear which parts of the code are really sharable.

Performance has improved a little as reading and writing of data from/to the sim has been optimized.

Discord username (if different from GitHub): cdr_maverick

Testing instructions

Full regression tests on the A32NX with full flights from c&d, runway and the air.
Also fuel handling (fueling, saving and loading of fuel per ATC ID) needs to be tested.

Things to look out for:

  • unusual thrust limits in the EWD
  • unusual commanded
  • unusual values or engine behaviour
  • FLX to CLB transistion (or other transisitons - but this is the critical one)
  • VNAV issues (e.g. incorrect times) in the flight plan after vnav calculations
  • any other possible situations where engine thrust and the use of data from the engine might be relevant (e.g. VNAV)

Same for the A380X but to a lesser extend as the fadec code is unfinished anyway.
As on the fuel page in the A380X the order of the tanks changed please also test and look at this.

How to download the PR for QA

Every new commit to this PR will cause a new A32NX artifact to be created, built, and uploaded.

  1. Make sure you are signed in to GitHub
  2. Click on the Checks tab on the PR
  3. On the left side, click on the bottom PR tab
  4. Click on the A32NX download link at the bottom of the page

@frankkopp frankkopp marked this pull request as ready for review April 12, 2024 12:54
Copy link
Member

@tracernz tracernz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've review the new changes (LGTM) and otherwise relying on the previous reviews. Good to go QA 👍 .

@frankkopp
Copy link
Member Author

I've review the new changes (LGTM) and otherwise relying on the previous reviews. Good to go QA 👍 .

Thanks - I'll update the test instruction to be more clear what should be tested.

@frankkopp frankkopp added Do Not Merge Not Ready For Testing Not ready for testing as still being discussed or developed. labels Apr 12, 2024
@frankkopp frankkopp removed Do Not Merge Not Ready For Testing Not ready for testing as still being discussed or developed. labels Apr 12, 2024
@frankkopp frankkopp self-assigned this Apr 12, 2024
@Watsi01
Copy link
Contributor

Watsi01 commented Apr 12, 2024

Non_QA Tester:

Did a full flight (EDDH-EDDM with full FMS programming)

  • Alt and speed calculation (FLP page) works now.
  • ETA and EFOB calculation works again.
  • Manipulating/deleting constraints works (calculation updates accordingly).
  • FLX works - correct %N1 when having a flx temp.
  • Throttle works again (there shall be thrust)
  • Graduate increase of thrust when going to CLB mode (having a low flx takeoff thrust) works.
  • TOC and TOD calc works

Everything looks good.

Passed!

@Saschl
Copy link
Contributor

Saschl commented Apr 15, 2024

Quality Assurance Report

Discord : saschl
Object of testing: #8622
Tier of Testing : 2
Date : 15.04.2024

Testing Process:

Full flight EDDS - LIRQ:

Engine start/shutdown working as expected.
FLX thrust working as expected and showing reasonable values.
VNAV calculations were reasonable.

Conducted takeoffs with FLX and TOGA, no issues found.
Observed transition from FLX to CLB phase. Thrust increase happens gradually, no issues found.

Checked if fuel is loaded and persisted properly when starting C&D. Could not see issues.

Negatives:
N/A

Testing Results:
Passed

Conclusions:
Working as expected

Media:

Screenshot 2024-04-15 105906

Microsoft.Flight.Simulator.2024-04-15.15-09-27.mp4

@2hwk 2hwk enabled auto-merge (squash) April 15, 2024 14:15
@2hwk 2hwk merged commit e618ed8 into flybywiresim:master Apr 15, 2024
7 checks passed
@frankkopp frankkopp deleted the fadec-rewrite branch April 15, 2024 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✔️ Done
Development

Successfully merging this pull request may close these issues.

5 participants