LagoLunatic / dsvrandom

Randomizer for DS Castlevania games
MIT License
80 stars 8 forks source link

Magic Seal locks removed regardless of settings #153

Closed Cloudmonkey98 closed 3 years ago

Cloudmonkey98 commented 3 years ago

DoS OnceAgainGivenFLESH - Spoiler Log.txt I didn't really think about it till I finished my run but I disabled the "Unlock Boss Doors" option and yet I didn't need Magic Seals nor were they distributed about the map(I wanted them to extend the search layers)

LagoLunatic commented 3 years ago

I disabled the "Unlock Boss Doors" option

Nope. It says it right there in your log - unlock_boss_doors. You must be remembering a different seed where you disabled that option.

LagoLunatic commented 3 years ago

Actually, apparently I made Unlock Boss Doors be forcibly enabled if you have map rando on? I guess there was some kind of incompatibility between locked doors and map rando, but it's been over 3 years since then, so I'm not sure if it's still broken considering how many things have changed in that time. I guess it would need to be tested.

Cloudmonkey98 commented 3 years ago

That explains everything, I thought I was going crazy and started testing and confirming it was being silently appended, thank you for checking, if nothing else, you should probably at least grey out and force tick the option like you do with things like Boss Souls(I checked, you're right, Map Rando forces Unlock Boss Doors) I would assume its a logic issue where it hamstrings Area distribution due to not being able to reassign what Seal goes to what Door or something? Also I wanna apologize for creating so many issues recently if I've been a bother, just sorta trying to feedback and help

LagoLunatic commented 3 years ago

It seems the main reason is because the item rando logic for where the seals are needed is tied to the doors on the outside, which get moved around by the map randomizer as rooms are reconnected. This could theoretically be fixable by having the logic itself change with the map. There are some more minor reasons for it too. Seems like too much work, so at least for now I'll simply grey out the option like you mentioned.