-
-
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(a380x/oans): EXIT missed, flags/crosses, visual fixes #9725
feat(a380x/oans): EXIT missed, flags/crosses, visual fixes #9725
Conversation
ec3c825
to
f334ad1
Compare
...e/flybywire-aircraft-a380-842/SimObjects/AirPlanes/FlyByWire_A380_842/model/A380_COCKPIT.xml
Show resolved
Hide resolved
# Conflicts: # fbw-a380x/src/systems/instruments/src/ND/OansControlPanel.tsx # fbw-a380x/src/systems/instruments/src/ND/instrument.tsx
# Conflicts: # .github/CHANGELOG.md
# Conflicts: # .github/CHANGELOG.md # fbw-a380x/src/systems/instruments/src/ND/OansControlPanel.tsx
Quality Assurance Tester Report Discord Username : utkrishtm Testing Process: Tested Airport: OMDB by iniBuilds, EHAM by FlyTampa, KJFK by iniBuilds Issue Fixes:
Context Menu Functionality:
BTV Testing:
OANS Performance Mode:
Circuit Breaker for ARPT NAV: Testing Results: Conclusions:
|
I forgot to comment, Over the last few days I flew a couple flights with this build. Right before takeoff I pull the ARPT NAV reset and it clears OANS from memory correctly. 10 minutes before TOD I go to set up my BTV, and the OANS is stuck on loading, the airport map never loads. To solve this, I had to relink navigraph. @utkrishtm did u ever encounter this? |
Yea so the map wouldn't load which is expected as it clears everything out. However if you choose the airport and click on Display Airport, it comes up. Make sure that the circuit breaker for ARPT NAV is clicked in before that. I did not test it the way you are. Try this and if it doesn't work ill test it again with what you did. |
Did you load an airport manually, as @utkrishtm described? |
I just tested how @aadee9940 did, and after clicking display airport again, it works fine. No need to relink Navigraph. |
@flogross89 @utkrishtm Yep so, when setting up BTV, I go to Plan Mode, zoom in till the OANS GUI pops up, Then going over to ARPT SEL tab, i had my departure and destination airport, KMIA, EGLL. After clicking on display airport for both and waiting upwards of 3-4 minutes, still nothing showed up. Hence stuck on loading. I did verify the ARPT NAV reset was in and not active. Another test I did was to simply monitor VRAM usage, when Oans is being used it can use upwards of 800-900mb, which is noticeable in the dev display. When I encountered this bug, the VRAM did not fill up as it was stuck on loading. After relinking navi, the vram usage went up and oans loaded in properly. If both of you do not encounter this issue, then something might me up with my navigraph, thats my guess. Sorry for the bother |
# Conflicts: # .github/CHANGELOG.md
That is still not correct FYI @flogross89. It should be above the other stuff. |
@flogross89 unfortunately I’ve just noticed that this PR seams to disable the opportunity to use BTV for no Navigraph users. The window where you used to put the brake distance in the past is now greyed out so you can’t insert the distance anymore. It looks like an interface issue to me. If you could fix that it’s would be great. :) |
@Ueeew Oh, that's not good. Do you have some more details? Does it disable the field always or "just" when the resets are pulled or performance mode unloaded the map? |
I'll redo the test without navigraph. Sorry missed that part |
No worries, also didn't think about it when writing the test instructions. Sometimes these things happen with complex functions :) |
Regarding this, I tested it across a real video of A380 at the same airport. and it looks the same. This is what i see in the sim at the same airport WMKK Not sure if I am missing anything |
I’ve just made one test flight. I was the last week not at home and when I came back and updated the aircraft BTV didn’t worked anymore. This PR is the only one which really has something to do with OANS and Btv so I assume that there is an issue. |
|
Ahh got it, thank you! I think I found the culprit. |
Actually it is working. I was just able to do it. 9725_BTVNoNaviWorking.mp4 |
If you look at it closely, in the sim the rubber line is obstructed by/behind the compass while on the real aircraft is not. That's what tracer was referring to |
Ahhh, that makes sense. Thanks! |
@tracernz @utkrishtm Indeed the ordering of the layers wasn't/is correct for the lubber line and the aircraft symbol. Fixed in #9849 |
Fixes #9672
Fixes #9514
Fixes #9798
Summary of Changes
Out of scope:
Screenshots (if necessary)
References
Additional context
Discord username (if different from GitHub): floridude
Testing instructions
Build: https://www.swisstransfer.com/d/9567e151-f0b7-4576-8350-ee84c62d4f08 (use this instead of the GitHub build)
OANS Performance Mode:
Monitor VRAM: Should go up considerably when loading airports, should go down to similar levels as before when staying at least 60 seconds out of the ZOOM range mode
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.