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.

Sign up for GitHub

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

Include triforce pieces in major item hints #2323

Open
wants to merge 3 commits into
base: Dev
Choose a base branch
from
+3 −17
Open

Include triforce pieces in major item hints #2323

Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
3e9cc8d
Include triforce pieces in major item hints
matthewkirby Oct 27, 2024
0b32a0d
Cleaned up duplicate code
matthewkirby Oct 28, 2024
a0a1f9c
Used scz shorthand and excluded gbk on lacs as its already hinted.
matthewkirby Oct 30, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
21 changes: 4 additions & 17 deletions Hints.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1149,24 +1149,11 @@ def get_important_check_hint(spoiler: Spoiler, world: World, checked: set[str])
for location in world.get_filled_locations():
region = HintArea.at(location).text(world.settings.clearer_hints)
if region == hint_loc:
if (location.item.majoritem
# exclude locked items
if ((location.item.majoritem or location.item.name in ('Biggoron Sword', 'Double Defense'))
and not location.locked
# exclude triforce pieces as it defeats the idea of a triforce hunt
and not location.item.name == 'Triforce Piece'
and not (location.name == 'Song from Impa' and 'Zeldas Letter' in world.settings.starting_items and 'Zeldas Letter' not in world.settings.shuffle_child_trade)
# Special cases where the item is only considered major for important checks hints
or location.item.name == 'Biggoron Sword'
or location.item.name == 'Double Defense'
# Handle make keys not in own dungeon major items
or (location.item.type in ('SmallKey', 'SmallKeyRing') and not (world.settings.shuffle_smallkeys == 'dungeon' or world.settings.shuffle_smallkeys == 'vanilla'))
or (location.item.type in ('HideoutSmallKey', 'HideoutSmallKeyRing') and not world.settings.shuffle_hideoutkeys == 'vanilla')
or (location.item.type in ('TCGSmallKey', 'TCGSmallKeyRing') and not world.settings.shuffle_tcgkeys == 'vanilla')
or (location.item.type == 'BossKey' and not (world.settings.shuffle_bosskeys == 'dungeon' or world.settings.shuffle_bosskeys == 'vanilla'))
or (location.item.type == 'GanonBossKey' and not (world.settings.shuffle_ganon_bosskey == 'vanilla'
or world.settings.shuffle_ganon_bosskey == 'dungeon' or world.settings.shuffle_ganon_bosskey == 'on_lacs'
or world.settings.shuffle_ganon_bosskey == 'stones' or world.settings.shuffle_ganon_bosskey == 'medallions'
or world.settings.shuffle_ganon_bosskey == 'dungeons' or world.settings.shuffle_ganon_bosskey == 'tokens'))):
and not (location.name == 'Song from Impa'
and 'Zeldas Letter' in world.settings.starting_items
and 'Zeldas Letter' not in world.settings.shuffle_child_trade)):
fenhl marked this conversation as resolved.
Show resolved Hide resolved
item_count = item_count + 1

checked.add(hint_loc + ' Important Check')
Expand Down
Loading