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): Correct flap/slats positions #8651
base: master
Are you sure you want to change the base?
fix(a380x): Correct flap/slats positions #8651
Conversation
9064aa9
to
9c59c77
Compare
fbw-common/src/wasm/systems/systems/src/simulation/update_context.rs
Outdated
Show resolved
Hide resolved
(cherry picked from commit bf94a44)
826f3be
to
5877d39
Compare
@tracernz Reworked some bits, your comments should now be addressed |
@@ -210,7 +211,7 @@ impl SlatFlapControlComputer { | |||
if context.is_in_flight() | |||
&& (context.indicated_airspeed().get::<knot>() | |||
< Self::ALPHA_SPEED_LOCK_IN_AIRSPEED_THRESHOLD_KNOTS | |||
|| context.angle_of_attack().get::<degree>() | |||
|| adirs_output.angle_of_attack(1).value().get::<degree>() |
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.
You should check the SSM before using the value. Should it just blindly always use ADR1, or should there be some selection logic to choose an ADR based on validity of the data/SSM?
fn set_angle_of_attack(&mut self, v: Angle) { | ||
self.angle_of_attack = Arinc429Word::new(v, SignStatus::NormalOperation); | ||
} | ||
} |
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.
This feels very weird. Don't we already have the stuff to mock the ADR in the test bed? @Gurgel100 will know better than me.
A380X only, no QA needed.
Summary of Changes
This should be an improvement to the current SFCC implementation for the A380X.
In scope:
Out of scope:
Screenshots (if necessary)
References
Additional context
No appropriate references for A380 FPPU, hence it's assumed the same FPPU as in the A320 is used.
Resulting FPPU angles:
Discord username (if different from GitHub): floridude
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.