-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
fix(a32nx/takeoff calculator): Accuracy update for takeoff calculator V2 speeds #9733
fix(a32nx/takeoff calculator): Accuracy update for takeoff calculator V2 speeds #9733
Conversation
Please add testing instructions |
Thanks for approving. I'll add them when it's ready for testing. It was supposed to be a draft, not ready for QA yet. |
Dev Team QA Report Discord Username : tracernz Testing Process: Testing Results: |
Fixes #[issue_no]
Summary of Changes
Changes to some of the V2 speed parameters for all flap configurations. This PR supersedes #9139.
Screenshots (if necessary)
This PR was initiated to address an issue brought up in the Discord QA channel regarding a takeoff calculation for LFBO (Toulouse/Blagnac airport). A screenshot of this original calculation is shown below:
The IRL FlySmart calculation is shown here:
After merging of this PR, the A32NX takeoff calculation will show:
Still not perfect, but much better. Making changes for better matches with the data always must be balanced with how they affect accuracy for other calculations. (Note: Small differences in some of the runway parameters were made to match the FlySmart calculation, but do not really change the conclusion.)
In PR #9139 , changes were proposed to provide a better match for some specific calculations at EDDB. A screenshot of the old calculation is shown below:
Here is the IRL FlySmart calculation:
And here is the A32NX calculation after this PR is merged:
A more complete comparison of the A32NX takeoff calculator to the IRL FlySmart calculator and the SimBrief takeoff calculator is shown in the attached file for a large number of example cases.
Takeoff Calculator Comparisons - IRL, A32NX, SimBrief.pdf
References
Additional context
Discord username (if different from GitHub):
Testing instructions
There are no coding changes in this PR, only changes to some constants used in the calculations, so it should not create any issues for the airplane. Feel free to give it a whirl and so a couple of calcs to verify I guess.
How to download the PR for QA
Every new commit to this PR will cause new A32NX and A380X artifacts to be created, built, and uploaded.