Skip to content
This repository has been archived by the owner on May 16, 2024. It is now read-only.

CJ4: 0.4.1 prerelease #235

Merged
merged 52 commits into from
Oct 12, 2020
Merged

CJ4: 0.4.1 prerelease #235

merged 52 commits into from
Oct 12, 2020

Conversation

dga711
Copy link
Member

@dga711 dga711 commented Oct 11, 2020

Fix Bugs!

TODO
Things we don't fix here we should create issues for after release

Audio (still on another branch)

  • Need Chime when playing seatbelt announcement
  • Chimes are played when setting battery master to on
  • Need to set States correctly for the groups so sounds dont stop when going external view and back in. See https://discordapp.com/channels/750764704175226992/764122525323362325/764904295618969600
  • Add passenger briefing announcements
  • Restrict only one announcement to play at once
  • Improvements to climate control knob sound
  • Fix passenger sounds stopping when switching to outside view
  • Fix chime playing when battery is turned on

Copy link
Contributor

@tscharlii tscharlii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Regarding 5579a87

A simple equality check does not work. Note the fixed length format of airway.icaos. It's not just a list of identifiers, but contains more information.
Index 0: Type (A=Airport, V=VOR, W or I = Waypoint or Intersection, etc.)
Index 1 and following: I assume that's the region
Index 7, 8, 9, 10, 11: Identifier, padded with spaces for shorter than 5 character waypoints

airway.icaos = ["VEB    NIK  ","WEB    DENUT","WEB    LUMEN","WEB    BULAM","WEB    DIBLI","WEG    RAPIX","WEG    SUNUP","WEG    TEBRA","WEG    KOPUL","WEG    UNSAD","WEG    NILON","WEG    GILDA","VEG    LAM  ","WEG    HEMEL","WLT    EKSEN","WLT    EKMIM","WLT    UPKUL","WLT    EMKUD","WLT    OSGED","WLT    PESOX","WLT    SIVKU","WLT    APSER","WLT    ETAMP","WLT    EMGIM","VLT    YAA  ","VLT    IST  ","WLT    FENER","WLB    VADEN","WLY    UTEKA","WLY    DONIV","WLY    PELOV","WLY    TISAK","WLY    TONDO","WED    RENKA","WED    GONBA","WED    STAUB","WED    MAMOR","WED    UNKUL","WED    UPALA","WED    INBED","WED    AMOSA","WED    IBAGA","WED    BARSU","WED    KOMIB","WED    TUSOS","WED    BIBEG"]

With the following code in line 448, I was able to add the NIK -> L610 -> LAM airway

let lastWaypointIcao = airway.icaos.find(icao => icao.substring(7, 12) === lastWaypointIdent.padEnd(5, " "));

Copy link
Contributor

@tscharlii tscharlii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Duplicate comment. Removed.

@dga711
Copy link
Member Author

dga711 commented Oct 12, 2020

Regarding [5579a87]

@tscharlii i see. i switched up icao and ident and didnt properly look into it because it was late ^^
i will check again and look into the vars while debugging. thanks for the heads up.
everything you have to do with waypoints feels like a hack job ;)

@mattnischan mattnischan merged commit 154de0f into main Oct 12, 2020
@mattnischan mattnischan deleted the CJ4-0.4.1-Fixes branch October 12, 2020 23:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.