-
-
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
feat(a380/pfd): FMA code cleanup & update possible fmas #9501
base: master
Are you sure you want to change the base?
Conversation
Are you sure ALT CRZ can be armed/ALT CRZ* can be engaged I thought it was ALT blue, ALT*, and then it turns into ALT CRZ. Maybe check with one of our autoflight gurus such as @lukecologne. |
|
Quality Assurance Tester Discord Username : alexdnzdmb Testing Process: Testing Results: Negatives: Conclusions: Media: |
@alexr4339 Please carefully re-read the testing instructions. This is a regression test only, the new modes are not hooked up to anything yet. |
I had coordinated with alex over discord after the fail and asked a retest (which now is a pass/PR commented has been edited after a new flight). The testing instructions were updated after the initial "fail" to make it more clear for the retest. |
Ah, great 👍 |
Fixes #[issue_no]
Summary of Changes
Cleanup reference to FINAL APP in PFD FMA file as it doesn't exist on the A380.
Updates possible FMA list with OP CLB blue, RWY blue, RWY + NAV blue, ALT CRZ * & ALT CRZ blue
(These are not hooked to anything as it require implementation on the simulink model).
Screenshots (if necessary)
References
Additional context
Discord username (if different from GitHub):
bruno_pt99
Testing instructions
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.