-
-
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
fix: enable transition from done phase to preflight phase #9664
base: master
Are you sure you want to change the base?
Conversation
Thanks a lot for the input 👍 I think the missing switch to the status page on FMS2 can be addressed in another PR, as this one is meant to allow for turnarounds in the first place. We will look into that for sure |
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.
looks good
@@ -139,6 +151,23 @@ export class FlightManagementComputer implements FmcInterface { | |||
// TODO remove this cyclic dependency, isWaypointInUse should be moved to DataInterface | |||
private dataManager: DataManager | null = null; | |||
|
|||
private readonly flightPhase = ConsumerSubject.create<FmgcFlightPhase>(null, this.flightPhaseManager.phase); |
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.
private readonly flightPhase = ConsumerSubject.create<FmgcFlightPhase>(null, this.flightPhaseManager.phase); | |
private readonly flightPhase = ConsumerSubject.create<FmgcFlightPhase>(this.sub.on('fmgc_flight_phase'), this.flightPhaseManager.phase); |
You can do this, if you pull private sub = this.bus.getSubscriber<FlightPhaseManagerEvents & MfdUIData>()
to the class methods, saves you two lines of setConsumer
calls
@Saschl i tested it and it works really good. There are you a few things left which I still need to check. I have just a small request for you, could you bring this "pull request" up to date again by merging the "master" file again into this "PR", so i can perform a full flight and test it again. Thank you |
3f8786b
to
29a2169
Compare
Quality Assurance Tester Report Discord Username : bruno_pt99 Testing Process:
Testing Results: Negatives: Conclusions: 2025-01-02.18-32-35.mp4 |
First I want to say that this is really nice and is already working quite good. But their are a few problems which I noticed while making 2 flights with a turnaround in between. I don’t know if this is related to your PR but I just wanted to list them up. But before starting to list up things which aren’t working yet as they should I wanted to say thank you for what you did so far. BTV reset is not correct (the BTV Rwy remains on ND) The trim shows an incorrect pre-trim value. 387 tons with a zfwcg of 34.4 and a cg of 35.5 show me a ths value of 33.0, which I should set for takeoff. then I have this exact 33.0 percent ths set and at takeoff role, as soon as I set my throttle above cl notch, the ECAM shows the warning. THS is not set for takeoff. Importantly, these things only showed up after landing(BTV bug and FMS 2 reset) and after turnaround ( everything else) |
Overall this looks really good. |
It does and it works really good too. |
Thanks for the feedback. The checklists are not supposed to reset themselves immediately , it takes 50 minutes for the automatic reset to happen if you didn't manually reset a checklist yourself (if its not doing this in the development version please create an issue). Regarding the other things which are not related to the FMS/MFD it would also be a good idea to create an issue as the original intent of this PR is to only fix the FMS fields not being editable across flights. |
Fixes #[issue_no]
Summary of Changes
Currently the plane never transitions to done phase, which prevented turnarounds from working properly as it was impossible to enter a new flightplan and performance data (the data is being cleared but the MFD fields are not editable). Added the logic to transition to preflight phase.
Not sure if this is the right way to approach this, suggestions welcome :)
Screenshots (if necessary)
References
Additional context
Discord username (if different from GitHub):
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.