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

Acc thr red correction as tested #309

Open
wants to merge 26 commits into
base: dev
Choose a base branch
from

Conversation

flinkeKoralle
Copy link

@flinkeKoralle flinkeKoralle commented Oct 18, 2023

acceleration altitude and thrust reduction input on PERF-TO corrected

Description of Changes

Once departure airport is chosen, the default company accel/thrRed is set.
The minimum value that can be inserted (manual entry) is the company default.
The company default value can be inserted in the specific livery aircraft.
E.g. DLH is 1000/1000 agl.
To Do:
The acceleration altitude is inserted in the Simple_C5 line of the mcdu.
A better solution would be to implement a split line for split entries on left and right side of mcdu.

Screenshots (optional)

Bugs fixed (if any)

Checklist:

  • My changes follow the Contributing Guidelines.
  • My changes implement realistic features.
  • Please have a main Developer test my changes before merging.

@flinkeKoralle flinkeKoralle marked this pull request as ready for review October 18, 2023 18:36
@flinkeKoralle flinkeKoralle reopened this Oct 18, 2023
@flinkeKoralle
Copy link
Author

Howto test it:
Insert Dep/Dest airport in INIT-A. ThrRed/Accel are inserted on default values.
They can be manually overridden.

@flinkeKoralle
Copy link
Author

PERFGA Page is also improved:
As GoAround Accel and ThrReduction is different from the T/O Accel/ThrRed, I introduced new properties.
It still needs some cleanup and checking.

@merspieler
Copy link
Collaborator

Hey, have you gotten around to do the cleanup you talked about?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants