-
-
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(cond): air conditioning failures #8178
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work
fbw-common/src/wasm/systems/systems/src/air_conditioning/acs_controller.rs
Outdated
Show resolved
Hide resolved
fbw-common/src/wasm/systems/systems/src/air_conditioning/acs_controller.rs
Outdated
Show resolved
Hide resolved
fbw-common/src/wasm/systems/systems/src/air_conditioning/acs_controller.rs
Outdated
Show resolved
Hide resolved
fbw-common/src/wasm/systems/systems/src/air_conditioning/acs_controller.rs
Outdated
Show resolved
Hide resolved
fbw-common/src/wasm/systems/systems/src/air_conditioning/acs_controller.rs
Outdated
Show resolved
Hide resolved
fbw-a380x/src/wasm/systems/a380_systems/src/air_conditioning/mod.rs
Outdated
Show resolved
Hide resolved
fbw-a380x/src/wasm/systems/a380_systems/src/air_conditioning/mod.rs
Outdated
Show resolved
Hide resolved
fbw-a32nx/src/wasm/systems/a320_systems/src/air_conditioning.rs
Outdated
Show resolved
Hide resolved
fbw-a32nx/src/wasm/systems/a320_systems/src/air_conditioning.rs
Outdated
Show resolved
Hide resolved
fbw-a32nx/src/wasm/systems/a320_systems/src/air_conditioning.rs
Outdated
Show resolved
Hide resolved
Thanks a lot @Gurgel100 and @Benjozork for the review!! All changes made following your suggestions. |
fbw-common/src/wasm/systems/systems/src/air_conditioning/acs_controller.rs
Outdated
Show resolved
Hide resolved
fbw-common/src/wasm/systems/systems/src/air_conditioning/mod.rs
Outdated
Show resolved
Hide resolved
fbw-a32nx/src/wasm/systems/a320_systems/src/air_conditioning.rs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work as always.
Behaves as expected is quite vague for testers, especially when it comes to PRs which add failures, maybe elaborate more upon this. Approved for QA with this in mind. |
I've added detailed instructions now for testing and resolved merged conflicts, thanks for the approval! |
QA Report Discord: alepouna Testing Process:
Testing Results: Notes: |
Summary of Changes
This PR introduces failures for the Air Conditioning System. In order to do this properly, the system has been modified slightly to more accurately model an A320 equiped with an ACSC. Other additions/changes are:
Failable items:
Other failures:
All associated ECAM warnings have been added as well.
Screenshots (if necessary)
References
Additional context
This PR will have conflicts with #8086, this one should be merged first as it has the fix for the temperature jumping when the valves are open and closed.
Discord username (if different from GitHub):
Testing instructions
What to look for in each of the failures:
I might’ve forgotten something in my instructions; a lot of these failures are interconnected so if you see something strange make sure to put it in the comments!
Validate that the air conditioning system (without any failures) works as before.
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.