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

fix(a380x/air cond): default start state of air cond system #8563

Merged
merged 4 commits into from
Mar 16, 2024

Conversation

mjuhe
Copy link
Contributor

@mjuhe mjuhe commented Mar 11, 2024

Summary of Changes

A380X only.

This PR improves the start state of the air conditioning system, essentially just changing the overhead temperature knobs to match standard procedures. It also adds the ability to select PURS SEL for the cabin temperature. This can later be added in the EFB (using LVar A32NX_COND_PURS_SEL_TEMPERATURE) to simulate flight attendants controlling the cabin temperature via the FAP. At the moment this LVar is set to 24 degrees.

It also adds a fix for the airflow determination when the number of passengers is still not entered in the FMS. Now it is adjusted for the maximum number of passengers (minimum before).

Screenshots (if necessary)

Screenshot 2024-03-10 202020

References

Additional context

Discord username (if different from GitHub): Commander Ga

Testing instructions

Check that the initial state of the overhead panel matches the screenshot above in all possible starting states.

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.

  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 either flybywire-aircraft-a320-neo or flybywire-aircraft-a380-842 download link at the bottom of the page

@2hwk 2hwk added QA A380 Only QA only for A380 required QA Ready to Test labels Mar 13, 2024
@2hwk 2hwk merged commit 6341eae into flybywiresim:master Mar 16, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
QA A380 Only QA only for A380 required QA Ready to Test
Projects
Status: ✔️ Done
Development

Successfully merging this pull request may close these issues.

2 participants