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
5k stars 1.05k forks source link

Altitude call out too late #2086

Closed bisinchi closed 2 years ago

bisinchi commented 3 years ago

Mod Version

Master

{ "built": "2020-11-24T08:08:38+00:00", "ref": "refs/heads/master", "sha": "b570de8c53cd5b52a21c81f8120bfa57d8c53830", "actor": "wpine215", "event_name": "push" }

Describe the bug Altitude callout (500 hundred, 400 hundred, 300 hundred, etc seem a bit late.

Altitude callout (500 hundred, 400 hundred, 300 hundred, etc seem a bit late. The audio seem triggered when the associated altitude is reached, whereas I believe that the audio should be completed at altitude crossing,

To Reproduce execute a normal landing

execute a normal landing and pay attention to altitude callouts. Practising circuits with latest master build, I encountered this. Not sure though if this was the case in previous master or release. 1. 2. 3. **Expected behavior**

audio should be completed at altitude crossing Actual behavior

audio start at altitude crossing, meaning when the sentence is finished, the aircraft is actually below that altitude References

Additional context

Was this working before/when did the issue start occurring?

Is this a problem in the vanilla unmodded game? Not sure.

Discord username (if different from GitHub):

BY REMOVING THIS LINE, I ATTEST THAT I HAVE FOLLOWED THE ISSUE TEMPLATE AND PROVIDED A TITLE, as well as searched for other possible duplicate issues. I acknowledge if this line is not removed and/or the issue template is not respected, my issue will be closed without warning

Benjozork commented 3 years ago

Please add a title and provide references for the behaviour you think is accurate.

ntlgr commented 3 years ago

The OP seems to be right. I have checked multiple videos on YT detailing the PFD on approach and it's always the case. Example: https://youtu.be/bjGT9aJU6jo

lukecologne commented 3 years ago

Documents say the callouts start a bit higher than the altitude you would expect, for example the 100 callout starts at 110 feet, or 50 feet starts a 53 feet.

bisinchi commented 3 years ago

Please add a title and provide references for the behaviour you think is accurate.

Sorry, I missed out on that, it's corrected.

Benjozork commented 3 years ago

Still no references.

bisinchi commented 3 years ago

Still no references. I'm trying to find documents that reference the callout issue. I think it is described in the FCOM of the aircraft, if anyone has access to it (Navigation/radio altimeter section).

krangerich commented 3 years ago

There seems to be no exact description in the FCOM. Here the corresponding part: image image

The phrase "in the detection zone" might be just a slight indication, that the callout starts before reaching the height

bisinchi commented 3 years ago

Yes, saw that too, was hoping to see more details in the FCOM. Seems like we need to have FWC spec to get the logics for triggering the sounds.

AirbusA320Pro commented 3 years ago

The OP is correct. IRL the Airbus needs to have completed saying to the word/phrase at or before the exact height. The reason for this, as it relates to decision making. If at the exact minimum decision height (DH) the audio only starts to speak, then the aircraft will be well below the intended height before a decision can be made.

A320PHX commented 3 years ago

Related to the altitude callouts, is anyone experiencing every altitude being called out at least twice in a row?

wpine215 commented 3 years ago

Related to the altitude callouts, is anyone experiencing every altitude being called out at least twice in a row?

Yes, I've had that issue as well. Probably warrants its own separate github issue if one doesn't already exist for it

bisinchi commented 3 years ago

I have the double callout issue on master downloaded on 1st December. I think this should be a separate issue to this one.

derl30n commented 3 years ago

cc @hotshotp can you address this?

hotshotp commented 3 years ago

cc @hotshotp can you address this?

No, it's all asobo logic for the call-outs and someone else will need to code them

beheh commented 3 years ago

While I can't share the exact reference, I'm aware of the following numbers:

In future the reworked FWC will implement this correctly, but for now we can probably still fix the old system if we want.

2hwk commented 3 years ago

I have a feeling this old issue is already addressed, if so then this can be closed.

beheh commented 3 years ago

@2hwk No, it's not. Our GPWS class still checks for the round altitudes, where it should be checking for the earlier ones I mentioned in an earlier comment. This will eventually be implemented by the new FWC, but somebody could take those numbers and plug it into the old GPWS class to get the correct behavior even without the new FWC.