-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
feat(a380x): BTV, ROW, ROP improvements #10372
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
base: master
Are you sure you want to change the base?
Conversation
add LIMITATIONS: LDG DIST AFFECTED, LDG PERF AFFECTED add BTV/ROW/ROP failure states to FCDC
- triple click when deactivation on ground or disarming in flight below 700ft RA - BTV fault conditions to Rust - ROW/ROP fault conditions to Rust
- add LGCIU bus publisher - refactor BtvPublisher to account for the above
- set airport when landing RWY detected by ROPS
… communication to Rust BTV from one place (in systems-host) still doesn't work
# Conflicts: # fbw-a380x/src/systems/systems-host/PseudoPRIM/BrakeToVacateDistanceUpdater.ts # fbw-common/src/systems/instruments/src/OANC/Oanc.tsx # fbw-common/src/systems/instruments/src/OANC/OansBrakeToVacateSelection.ts # fbw-common/src/systems/oans/OansMapProjection.ts # fbw-common/src/systems/shared/src/publishers/OansBtv/FmsOansPublisher.ts
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.
Some minor comments, also, you forgot to take it out of draft I believe
fbw-a380x/src/systems/systems-host/CpiomC/FlightWarningSystem/FwsCore.ts
Show resolved
Hide resolved
fbw-a380x/src/systems/systems-host/CpiomC/FlightWarningSystem/FwsCore.ts
Outdated
Show resolved
Hide resolved
…I reversion when different runway is approached
tracernz
left a comment
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.
Just reviewing the ARINC bus publishers only.
fbw-common/src/systems/shared/src/publishers/ArincBus/RaBusPublisher.ts
Outdated
Show resolved
Hide resolved
fbw-common/src/systems/shared/src/publishers/ArincBus/LgciuBusPublisher.ts
Show resolved
Hide resolved
fbw-common/src/systems/shared/src/publishers/ArincBus/LgciuBusPublisher.ts
Outdated
Show resolved
Hide resolved
fbw-common/src/systems/shared/src/publishers/OansBtv/BtvPublisher.ts
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.
Thanks for the fixes! I scanned over the rest and note a couple of small suggestions. Otherwise LGTM with the import that's failing the build fixed up.
fbw-a380x/src/systems/systems-host/CpiomC/FlightWarningSystem/FwsCore.ts
Outdated
Show resolved
Hide resolved
BravoMike99
left a comment
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.
Some more minor things
fbw-a380x/src/systems/systems-host/CpiomC/FlightWarningSystem/FwsCore.ts
Show resolved
Hide resolved
fbw-a380x/src/systems/systems-host/CpiomC/FlightWarningSystem/FwsCore.ts
Outdated
Show resolved
Hide resolved
BravoMike99
left a comment
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.
Some minor things
fbw-a380x/src/systems/systems-host/CpiomC/FlightWarningSystem/FwsCore.ts
Outdated
Show resolved
Hide resolved
Jonny23787
left a comment
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.
Just for future consistency
fbw-a380x/src/systems/systems-host/CpiomC/FlightWarningSystem/FwsAbnormalSensed.ts
Outdated
Show resolved
Hide resolved
fbw-a380x/src/systems/systems-host/CpiomC/FlightWarningSystem/FwsAbnormalSensed.ts
Outdated
Show resolved
Hide resolved
# Conflicts: # fbw-a380x/src/systems/systems-host/CpiomC/FlightWarningSystem/FwsCore.ts # fbw-a380x/src/systems/systems-host/CpiomC/FlightWarningSystem/FwsLimitations.ts
|
Quality Assurance Tester Report Discord Username : jonny_23 Testing Process:
Testing Results: Negatives:
Conclusions: |


Fixes #9705
Fixes #10043
Summary of Changes
Note: The arming/disarming and reversion logic might not be 100% accurate yet, but it brings us closer to where we want to be.
Screenshots (if necessary)
References
Additional context
Discord username (if different from GitHub): floridude
Testing instructions
Regression tests:
New features test:
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.