vg-json-data / sm-json-data

JSON representations of Super Metroid Game Data
Other
23 stars 11 forks source link

More Logic Updates #1529

Closed osse101 closed 1 month ago

osse101 commented 1 month ago

Maybe its too early to put flashSuitChecked on non-trivial and grapple teleport strats?

kjbranch commented 1 month ago

Just saw your main message asking about flash suits. I think its fine to include them if youre sure on them. I havent ever tested it on a grapple teleport, but would assume it works.

blkerby commented 1 month ago

Yeah, currently the randomizer is ignoring all the flash suit logic; e.g., "canSpikeSuit" is an ignored tech and won't show up on the Generate page. I think we won't use it until we have at least some reasonable level of coverage of strats for carrying it around; and at that point there's some work I'll need to do in the randomizer to allow it to account for it. So in the meantime, there's no harm in adding "flashSuitChecked" everywhere that it is known to be correct.

It might make sense to add a tech "canTrickyCarryFlashSuit" or something for where a strat becomes significantly harder to do while carrying a flash suit; and that probably that would go into Insane? We could define a helper "h_canTrickyCarryFlashSuit" as {"or": [{"noFlashSuit": {}}, "canTrickyCarryFlashSuit"]}