sporchia / alttp_vt_randomizer

ALttP VT Randomizer and API
https://alttpr.com/
MIT License
315 stars 147 forks source link

Basic inverted keysanity generation failures #913

Closed krelbel closed 1 year ago

krelbel commented 3 years ago

Settings

All default, except: Item Placement: Basic Dungeon Item Shuffle: Keysanity World State: Inverted

Description of problem:

Selecting basic+inverted+keysanity on alttpr.com (either in the normal randomizer or customizer) frequently fails to generate seeds. This is probably because of how few locations are in early inverted spheres, increasing the likelihood of running out of locations to place items that can get past the high key + health requirements added by basic placement + keysanity. For reference, no locations in TT are accessible without a sword (unless swordless), 7 hearts, and a bottle: https://github.com/sporchia/alttp_vt_randomizer/blob/master/app/Region/Inverted/ThievesTown.php#L27 . One reasonable way to increase generation rate with this combination of options would be to increase starting health in basic inverted.

krelbel commented 3 years ago

One reasonable way to increase generation rate with this combination of options would be to increase starting health in basic inverted.

It seems like interactions with the customizer's ability to set custom starting health might get a little weird if we followed this suggestion. MichaelK on discord mentioned a much better solution to this problem: adjusting the requirements (at least for TT/SW, maybe others) to open up certain easy to reach chests before hitting the health/bottle/weapon basic logic gates. I think reducing the basic protection to the back 4 chests in TT and only the mothula chest in SW would be a good start.

sporchia commented 1 year ago

Currently going to close this while we consider v32 options to solve the problem. It is duly noted and will be something we assure will be addressed in the new graph based code.