search

flybywiresim / aircraft

The A32NX & A380X Project are community driven open source projects to create free Airbus aircraft in Microsoft Flight Simulator that are as close to reality as possible.
https://flybywiresim.com
GNU General Public License v3.0
5.14k stars 1.12k forks source link

EFIS QNH Range Incorrect #8397

Open tshomas opened 10 months ago

tshomas commented 10 months ago

Aircraft Version

Development

Build info

{
    "built": "2024-01-08T21:35:42+00:00",
    "ref": "refs/heads/master",
    "sha": "4230ccef32e6c711e513153b659d7a87428a6739",
    "actor": "Saschl",
    "event_name": "manual",
    "pretty_release_name": "master:4230ccef",
    "version": "v0.12.0-dev.4230cce"
}

Describe the bug

Reported by ThePilotGamer via discord, the QNH range of the EFIS dial is limited below the actual possible range. This is separate to the mcdu min and max ranges for data entry.

Expected behavior

PXL_20240109_210603143

Steps to reproduce

Set QNH dial minimum and maximum ranges.

References (optional)

No response

Additional info (optional)

No response

Discord Username (optional)

shomas

goochs commented 10 months ago

I'd like to try to tackle this, could it be assigned to me?

tshomas commented 10 months ago

Always feel free to go for it! If you get some code working, make a quick PR that references this issue and it'll all assign up nicely 👍

tracernz commented 10 months ago

I'd like to try to tackle this, could it be assigned to me?

Sorry, it's not possible to fix this without first fixing the autopilot to use our ADR ARINC words rather than using the sim altimeters, then implementing the FCU bus to send the QNH properly to the ADRs, and shifting any remaining users of the sim altimeters. The autopilot is the sticking point.

goochs commented 10 months ago

Sorry, it's not possible to fix this without first fixing the autopilot to use our ADR ARINC words rather than using the sim altimeters, then implementing the FCU bus to send the QNH properly to the ADRs, and shifting any remaining users of the sim altimeters. The autopilot is the sticking point.

I only recently got the chance to sit down and look at the code and that tracks with what I was thinking, it's much more than I was initially imagining. Thanks for the confirmation tracernz!

tracernz commented 10 months ago

I'd say https://github.com/flybywiresim/aircraft/pull/7587 is probably a pre-req for this.

lukecologne commented 2 months ago

This depends on the FCU standard, since a certain FCU standard, the maximum selection was changed to 1100hPa. But, since this FCU standard seems to already be relatively old, it's safe to assume it's installed on all neos. This will be fixed in #7587

tracernz commented 2 days ago

This is not fixed yet. We have to stop using the sim altimeters for that.