mmp / vice

Virtual air traffic control simulator
https://pharr.org/vice
GNU General Public License v3.0
18 stars 25 forks source link

CIFP: Invalid output if the altitude restriction is a range #186

Closed Umoxfo closed 2 months ago

Umoxfo commented 2 months ago

Since the ARINC 424 specification and the FAA CIFP amendment do not define the order of the altitude fields, it should not be assumed that they are always described in a specific order.

Example:

PS C:\Program Files (x86)\Vice> ./vice.exe -routes KSMF
PS C:\Program Files (x86)\Vice> STARs:
[...]
SLMMR5.DONNR: DONNR CARVV/a22000-20000 SLMMR/a17000-13000/s280 POZUR/a8000+
SLMMR5.WEBGO: WEBGO/a22000-20000 SLMMR/a17000-13000/s280 POZUR/a8000+
SLMMR5.ANAHO: ANAHO WEBGO/a22000-20000 SLMMR/a17000-13000/s280 POZUR/a8000+
SLMMR5.CARVV: CARVV/a22000-20000 SLMMR/a17000-13000/s280 POZUR/a8000+
SLMMR5.CRASY: CRASY FIORE CARVV/a22000-20000 SLMMR/a17000-13000/s280 POZUR/a8000+
SLMMR5.RWY17B: POZUR/a8000+ TENCO/a4000/flyover/h255
SLMMR5.RWY35B: POZUR/a8000+ RICEE PKAAY/a6000+/s210 THANN BANZA/a4000/flyover/h168
SUUTR4.NURAY: NURAY KAELA/a28000+ SUUTR/a26000-24000/s280 FOOLZ/a21000 GLDDD/a19000- WRAPS/a17000- PNNEN/a16000-12000 SLUIC
SUUTR4.RWY17B: SLUIC AMADR/a9000 NGGET/a9000-7000 CHNEL/a5000+ OVOME/a4000/s210/flyover/h348
[...]
mmp commented 2 months ago

Thanks for reporting! Fixed now.