Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Botwoon Energy Pit entrances are swapped #20

Open
makamys opened this issue Dec 5, 2022 · 4 comments
Open

Botwoon Energy Pit entrances are swapped #20

makamys opened this issue Dec 5, 2022 · 4 comments

Comments

@makamys
Copy link

makamys commented Dec 5, 2022

The randomizer thinks that Botwoon_Energy_Pit_TS1 is the top-left entrance of the room and Botwoon_Energy_Pit_TS2 is the top-right one, but in-game it's the other way around.

Example

In the seed generated like so:

python3 door_rando_main.py --clean sm.smc --seed "Coven, Squeept, Mella, Mella" --graph --create sm_rando.smc

Botwoon Energy Pit's connections look like this in the spoiler graph:

But the connections observed in-game are

Butterfly Sand Pit bottom -> Botwoon Energy Pit top-right -> Botwoon Energy Pit bottom-right -> Botwoon Energy Farm top

and

Botwoon Energy left pit -> Botwoon Energy Pit top-left -> Botwoon Energy Pit bottom-left -> East Sand Hole top

which makes the spoiler log's following route to the Morph Ball impossible:

... -> Butterfly_Sand_Pit_BS -> Botwoon_Energy_Pit_TS1 -> Botwoon_Energy_Pit_BS1 -> East_Sand_Hole_TS -> ...
@aremath
Copy link
Owner

aremath commented Dec 8, 2022

Thanks for bringing this up. I'll look into it (probably an error in encoding/dsl/door_defns.txt).

@aremath
Copy link
Owner

aremath commented Dec 8, 2022

Are you sure you're using the latest version? I checked the door_defns for Botwoon_Energy_TS1 and TS2 (the doors entering botwoon energy pit) and they are correct. I remember changing this file to fix this issue a while ago, but it's definitely possible that there's still some mistake.

@makamys
Copy link
Author

makamys commented Dec 8, 2022

I was using the latest version at the time (287bae3).

@aremath
Copy link
Owner

aremath commented Dec 8, 2022

Alright, I will look into this further.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants