feat(cpcu): Update low diff pressure warning trigger#10484
feat(cpcu): Update low diff pressure warning trigger#10484FozzieHi wants to merge 2 commits intoflybywiresim:masterfrom
Conversation
Co-authored-by: lukecologne <lukasmichels00@gmail.com>
|
|
||
| let activation_condition = time_to_delta_p_zero_secs < 90. | ||
| && time_to_delta_p_zero_secs < time_to_cabin_landing_secs + 30. | ||
| && altitude_above_landing_ft > 3000. |
There was a problem hiding this comment.
Sorry, I missed this in my earlier comment. Of course the alt above landing condition must be removed here, as otherwise the warning would disappear below 3000ft above landing alt.
|
I apologize for barging into the github uninvited, but I am able to reproduce the pressurization issues almost every time at my home airport KRDU, with very simple departure from 23R, heading straight out to 8000 ft, and then turning back to join approach DMESTR3 and ILS23R. When descending thru 4000ft, the packs, bleed and press starts to flip from 0 to 4000ft, randomly. Just in case you need possibly consistent reproduce scenario. Latest dev, no other mods or sceneries. |
|
@rsimac The problem you're experiencing is probably fixed by 7a99303 already in master (although I'm not sure that would cover the flipping you've experienced, that seems like another bug, it should solve the master caution), give it a try and let us know in Discord. This PR is just cleaning up the initial trigger to closer match the real aircraft. |
|
Oh yes the master caution is 'fixed', there is none anymore. Posted a video already. Thought the pack flipping is part of this issue. Thanks. |
|
Glad to know the master caution is fixed at least. This PR only changes the trigger for the master caution though, can you raise a separate GitHub issue for your problem? |
3 participants
Summary of Changes
Follow-up from #10477, very early POC PR to change the conditions of the low diff pressure bit in the CPCU to reflect that of the FCOM and AMM. Still very much a draft, and I have completely messed with the styling of the Rust tests...
Screenshots (if necessary)
References
Additional context
Discord username (if different from GitHub): fozzie
Testing instructions
It's going to be difficult to test the true positive outcome. I've tested it by manually opening the outflow valve on a descent whilst below 9,000ft and looking at sim vars. You could also enter into a very steep dive, which is the likely cause during service.
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.