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.

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

Bug: Heretic logic for E5M4 Silver Shield 2 (Kakis region) is NOT green OR yellow, but just yellow. #4662

Open
PointMeAtTheDawn opened this issue Feb 18, 2025 · 2 comments · May be fixed by #4677
Labels
is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs.

Comments

@PointMeAtTheDawn
Copy link

What happened?

I'm not familiar enough with the rules to be able to propose the proper fix across regions, but the northwest area with the silver shield 2 has no relevance to the green key. Because of https://github.com/ArchipelagoMW/Archipelago/blob/main/worlds/heretic/Rules.py#L626 (or L633?) being ORs, this generated a softlock for our multiworld.

Thanks!

What were the expected results?

The SS2 and surrounding items require the yellow key, green is insufficient.

Software

Website

@ScipioWright ScipioWright added the is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs. label Feb 18, 2025
@ScipioWright
Copy link
Collaborator

@KScl

@KScl
Copy link
Contributor

KScl commented Feb 18, 2025

Logic appears to be written as if the Wings of Wrath in the opening room is reachable by the player, but in reality it is too high up to reach until after obtaining both Yellow and Green keys.

There are a few logic issues (this being one of them) across the id Tech 1 games that I plan to fix in a later PR.

@KScl KScl linked a pull request Feb 23, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants