Skip to content

Fix misc. location hints not being counted as always hints #2391

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

Merged
fenhl merged 3 commits into from
Nov 26, 2025
Merged

Fix misc. location hints not being counted as always hints #2391

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
34c86d0
Fix misc. location hints not being counted as always hints
fenhl Mar 20, 2025
f659e8a
Fix location name being used instead of location
fenhl Mar 20, 2025
fa8f63e
Account for non-location names in checked_always_locations
fenhl Mar 20, 2025
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
37 changes: 16 additions & 21 deletions Hints.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1278,20 +1278,10 @@ def build_bingo_hint_list(board_url: str) -> list[str]:
return hints


def always_named_item(world: World, locations: Iterable[Location]):
for location in locations:
if location.item.name in bingoBottlesForHints and world.settings.hint_dist == 'bingo':
always_item = 'Bottle'
else:
always_item = location.item.name
if always_item in world.named_item_pool and world.settings.world_count == 1:
world.named_item_pool.remove(always_item)


def build_gossip_hints(spoiler: Spoiler, worlds: list[World]) -> None:
from Dungeon import Dungeon

checked_locations = dict()
checked_locations = {}
# Add misc. item hint locations to "checked" locations if the respective hint is reachable without the hinted item.
for world in worlds:
for location in world.hinted_dungeon_reward_locations.values():
Expand Down Expand Up @@ -1349,16 +1339,16 @@ def build_gossip_hints(spoiler: Spoiler, worlds: list[World]) -> None:
item_world = location.world
if item_world.id not in checked_locations:
checked_locations[item_world.id] = {}
mark_checked(checked_locations[item_world.id], location.name)
mark_checked(checked_locations[item_world.id], location.name, CheckedKind.ALWAYS)

# Build all the hints.
for world in worlds:
world.update_useless_areas(spoiler)
build_world_gossip_hints(spoiler, world, checked_locations.pop(world.id, None))
build_world_gossip_hints(spoiler, world, checked_locations.pop(world.id, {}))


# builds out general hints based on location and whether an item is required or not
def build_world_gossip_hints(spoiler: Spoiler, world: World, checked_locations: Optional[dict[HintArea | str, set[CheckedKind]]] = None) -> None:
def build_world_gossip_hints(spoiler: Spoiler, world: World, checked_locations: dict[HintArea | str, set[CheckedKind]]) -> None:
world.barren_dungeon = 0
world.woth_dungeon = 0

Expand All @@ -1368,9 +1358,6 @@ def build_world_gossip_hints(spoiler: Spoiler, world: World, checked_locations:
search.spot_access(world.get_location(stone.location))
and search.state_list[world.id].guarantee_hint())

if checked_locations is None:
checked_locations = {}

stone_ids = list(gossipLocations.keys())

world.distribution.configure_gossip(spoiler, stone_ids)
Expand Down Expand Up @@ -1491,8 +1478,6 @@ def build_world_gossip_hints(spoiler: Spoiler, world: World, checked_locations:
mark_checked(checked_locations, first_location.name, CheckedKind.ALWAYS)
mark_checked(checked_locations, second_location.name, CheckedKind.ALWAYS)

always_named_item(world, [first_location, second_location])

if hint.name in world.hint_text_overrides:
location_text = world.hint_text_overrides[hint.name]
else:
Expand All @@ -1514,8 +1499,6 @@ def build_world_gossip_hints(spoiler: Spoiler, world: World, checked_locations:
location = world.get_location(hint.name)
mark_checked(checked_locations, hint.name, CheckedKind.ALWAYS)

always_named_item(world, [location])

if location.name in world.hint_text_overrides:
location_text = world.hint_text_overrides[location.name]
else:
Expand Down Expand Up @@ -1572,6 +1555,18 @@ def build_world_gossip_hints(spoiler: Spoiler, world: World, checked_locations:

# Add user-specified hinted item locations if using a built-in hint distribution
# Raise error if hint copies is zero
for location_name, kinds in checked_locations.items():
if CheckedKind.ALWAYS in kinds:
try:
location = world.get_location(location_name)
except KeyError:
continue
if location.item.name in bingoBottlesForHints and world.settings.hint_dist == 'bingo':
always_item = 'Bottle'
else:
always_item = location.item.name
if always_item in world.named_item_pool and world.settings.world_count == 1:
world.named_item_pool.remove(always_item)
if len(world.named_item_pool) > 0 and world.hint_dist_user['named_items_required']:
if hint_dist['named-item'][1] == 0:
raise Exception('User-provided item hints were requested, but copies per named-item hint is zero')
Expand Down