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

New Item : Bronze scale #2224

Draft
wants to merge 3 commits into
base: Dev
Choose a base branch
from
Draft

Conversation

GSKirox
Copy link
Collaborator

@GSKirox GSKirox commented May 22, 2024

Inspired by SoH's implementation.

This draft PR requires some help for logic and testing in general. All help appreciated !

This adds a new first progressive scale in the item pool : the bronze scale. Until the player finds it, Link will automatically respawn if he goes fully in water, unless Iron Boots are put on.
The respawn will be at the last entrance, unless that entrance is underwater. All of these have been special cased to have a respawn nearby on a safe place to avoid softlocks.

Water Temple has two special cases :

  • The entrance won't make Link respawn, because putting Link up to a safe place would lead to softlocks since you can't savewarp out of a dungeon. We'll say it's magic water 😅
  • The river room has a special nearby respawn, out of mercy so that players don't get thrown back to the temple entrance.

The respawns don't lose any health, to make it play nice with high damage multiplier setting.

On the UI side, i added a 0 on the scale on the file select screen if the bronze scale setting is on and the player has no scale.
This would maybe also need a marker in the equipment pause screen. Not sure.

Logic has been implemented as best as i could, but that's the part where this PR needs the most help with.

@fenhl fenhl added Type: Enhancement New feature or request Status: Help Wanted Extra attention is needed Component: Logic Non-trivial changes to the JSON logic files Component: ASM/C Changes some internals of the ASM/C libraries Status: Needs Review Someone should be looking at it Component: Setting specific to setting(s) Status: Needs Testing Probably should be tested Status: Under Consideration Developers are considering whether to accept or decline the feature described Changes Item Table Adds/removes items from the item table. Allows coordination of multiple PRs which add to the table. labels May 22, 2024
@fenhl

This comment was marked as resolved.

@GSKirox

This comment was marked as resolved.

@ktspeedruns
Copy link

Instead of making an exception for Water Temple entrance, why not move the spawn point so you wouldn't be in the water, and also change the IDs of the polygons at the edge going up the exit so you don't even need to go into the water to exit Water Temple? Or, maybe even add Ruto at the beginning, and if you talk to Ruto she asks "Do you want to leave the temple?" and if you say yes, you spawn outside the Temple (or on Lake Hylia warp pad if it would put you in Lake Hylia)?

@GSKirox
Copy link
Collaborator Author

GSKirox commented May 24, 2024

I'd rather not modify the game scenes directly, i feel like that should be a last resort solution.
Also this is a lot more complex hacking, while the current exception is just a single line of code.

@Pepe20129
Copy link

Hi! I'm the one that made this for Ship.
2 things:

  • There's an entrance that I missed that softlocks you (getting caught by the gerudos as child)
  • In Ship's implementation, the way it deals with water temple is that it just kicks you out if you void due to swimming without bronze scale (taking into account entrance rando), I think the same should be done here (or at least something that has the same ramifications for logic)

@shirosoluna
Copy link

leaving bombchu bowling or other areas that affect the B button will put a stick on B. Just throwing it out there that if ER is not addressed with this such as having bowling in water temple then leaving but without bronze scale you would have temp B.
So perhaps this PR could also address the existing temp B issues that are unresolved in ER seeds since it would see it more often.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes Item Table Adds/removes items from the item table. Allows coordination of multiple PRs which add to the table. Component: ASM/C Changes some internals of the ASM/C libraries Component: Logic Non-trivial changes to the JSON logic files Component: Setting specific to setting(s) Status: Help Wanted Extra attention is needed Status: Needs Review Someone should be looking at it Status: Needs Testing Probably should be tested Status: Under Consideration Developers are considering whether to accept or decline the feature described Type: Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants