Archipelago icon indicating copy to clipboard operation
Archipelago copied to clipboard

LTTP: Move more options to options system

Open alwaysintreble opened this issue 3 years ago • 3 comments
trafficstars

This moves over a bunch of options:

  • The remaining logic options. logic is now glitches_required
  • Enemy options
  • Prize shuffle
  • Triforce options
  • Medallion options (with a fix for random default value if not defined in the YAML)
  • Dungeon counter mode.
  • mode renamed to world_state to be more clear and moved over
  • Item pool and functionality
  • Sprite pool is now an optionlist

Framework for other sprite options is present but not finished. Actual sprite is ready and will be a textchoice when #728 is merged. RandomSprite I still haven't decided the best course as far as naming convention goes. As far as implementation planning on this being split into a togle and option list as all of the options can be enabled at the same time.

alwaysintreble avatar Aug 10 '22 23:08 alwaysintreble

Should address all of the missing docstrings from #823

alwaysintreble avatar Aug 10 '22 23:08 alwaysintreble

  File "/worlds/alttp/Rom.py", line 1852, in apply_rom_settings
    {'normal': 0x00, 'hide_goal': 0x01, 'hide_required': 0x02, 'hide_both': 0x03}[triforcehud]
KeyError: 'always_show'

black-sliver avatar Aug 16 '22 21:08 black-sliver

It's been a while, admittedly, but we agreed that goal should be the last option moved over so that people realize that options are missing.

Berserker66 avatar Oct 02 '22 16:10 Berserker66