LaneDibello / Kotor-Randomizer

The Kotor Randomizer. A randomizer for Star Wars: Knights of the Old Republic. Intended for challenge runs and general fun.
4 stars 1 forks source link

Davik's Estate hid a star map with one-time transitions ignored #69

Closed glasnonck closed 3 years ago

glasnonck commented 3 years ago

In one of Indy's recent runs, Davik's Estate led directly to the Maanan Star Map even though he had enabled Ignore Single-Use Transitions. This should never happen.

Upon investigation, the issue appears to be that the IsReachable function that is used to test if certain goals can be completed is converting the original warp code to the randomized code even though the Reachable dictionary is expecting an original warp code as the key.

The solution is to simply use the original code to determine reachability. Some improved summary comments might also help to avoid this confusion in the future.