mracsys / tootr

Entrance and Location Tracker for OoT Randomizer
MIT License
5 stars 6 forks source link

Possible Data Issue in gerudo_fortress.json #45

Closed dcavoulacos closed 8 months ago

dcavoulacos commented 10 months ago

I was looking through the entrance data for Gerudo Fortress, and I think there might be some mixup in the data between GF Lower Vines -> Hideout 4 Torch Jail U Tunnel and Gerudo Fortress by GTG -> Hideout 4 Torch Jail L Tunnel:


"GF Lower Vines -> Hideout 4 Torch Jail U Tunnel": {
    "area": "Gerudo Fortress",
    "reverse": "Hideout 4 Torch Jail U Tunnel -> GF Lower Vines",
    "oneWay": false,
    "oneWayArea": "",
    "connector": "Hideout 4 Torch Jail L Tunnel -> Gerudo Fortress by GTG",
    "isReverse": false,
    "alias": "1F Near Toll Gate",
    "reverseAlias": "from 1F Near Toll Gate",
    "tag": "4 Torch Jail U Tunnel",
    "tagRep": true,
    "tagRules": [],
    "enableTag": true,
    "type": "hideoutInterior",
    "lKey": "",
    "aLink": "",
    "userALink": "",
    "eLink": "",
    "userELink": "",
    "oneWayELink": "",
    "userOneWayELink": "",
    "shuffled": false
},
"Gerudo Fortress by GTG -> Hideout 4 Torch Jail L Tunnel": {
    "area": "Gerudo Fortress",
    "reverse": "Hideout 4 Torch Jail L Tunnel -> Gerudo Fortress by GTG",
    "oneWay": false,
    "oneWayArea": "",
    "connector": "Hideout 4 Torch Jail U Tunnel -> GF Lower Vines",
    "isReverse": false,
    "alias": "2F North Vines",
    "reverseAlias": "from 2F North Vines",
    "tag": "4 Torch Jail L Tunnel",
    "tagRep": true,
    "tagRules": [],
    "enableTag": true,
    "type": "hideoutInterior",
    "lKey": "",
    "aLink": "",
    "userALink": "",
    "eLink": "",
    "userELink": "",
    "oneWayELink": "",
    "userOneWayELink": "",
    "shuffled": false
}```

At the very least, the aliases are mixed up; `1F Near Toll Gate` should match `Gerudo Fortress by GTG -> Hideout 4 Torch Jail L Tunnel`
mracsys commented 10 months ago

You're right, the aliases are backwards. Thanks for reporting it!

3pacgangstarap commented 9 months ago

added a quick fix for this in #46

mracsys commented 8 months ago

Merged with #46