Skip to content

Allow big poe soul collect on Epona#2377

Merged
fenhl merged 2 commits into
OoTRandomizer:Devfrom
GSKirox:big_poe_riding
Jun 3, 2025
Merged

Allow big poe soul collect on Epona#2377
fenhl merged 2 commits into
OoTRandomizer:Devfrom
GSKirox:big_poe_riding

Conversation

@GSKirox

@GSKirox GSKirox commented Feb 20, 2025

Copy link
Copy Markdown
Collaborator

Small QoL change to allow collecting a big poe soul while still riding Epona.
The collider of the soul actor has been changed to interact with anything that has an appropriate collider, but there is also additional protection to ensure that only either Link, or Epona when Link is riding, will trigger the textbox.

2025-02-20.18-53-33.mp4

@GSKirox GSKirox added Type: Enhancement New feature or request Component: ASM/C Changes some internals of the ASM/C libraries 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 Racing Impact Changes a mechanic in a way that impacts the balance of competitive racing. labels Feb 20, 2025
@r0bd0g

r0bd0g commented Feb 20, 2025

Copy link
Copy Markdown

Potential scenario, all of your bottles are full and then you run into it accidentally when you meant to get off and dump a bottle.

@flagrama

flagrama commented Feb 20, 2025

Copy link
Copy Markdown

Potential scenario, all of your bottles are full and then you run into it accidentally when you meant to get off and dump a bottle.

Skill issue. Can do the same thing as just Link.

@GSKirox

GSKirox commented Feb 21, 2025

Copy link
Copy Markdown
Collaborator Author

It's actually kinda hard to do it accidentally since you have to stop Epona on top of it

@fenhl fenhl removed the Status: Under Consideration Developers are considering whether to accept or decline the feature described label Mar 23, 2025

@rrealmuto rrealmuto left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good other than the one comment I made.

Comment thread ASM/src/big_poe.asm Outdated
@GSKirox

GSKirox commented May 20, 2025

Copy link
Copy Markdown
Collaborator Author

Implemented Rob's suggestion by using 0x34(sp) to grab Playstate

@fenhl fenhl requested a review from rrealmuto May 20, 2025 06:42
@fenhl fenhl removed the Status: Needs Review Someone should be looking at it label Jun 3, 2025
@fenhl

fenhl commented Jun 3, 2025

Copy link
Copy Markdown
Collaborator

Rob says this looks good from the review side.

@fenhl fenhl removed the Status: Needs Testing Probably should be tested label Jun 3, 2025
@fenhl fenhl added this to the next milestone Jun 3, 2025
@fenhl fenhl merged commit 3716f89 into OoTRandomizer:Dev Jun 3, 2025
3 checks passed
@GSKirox GSKirox deleted the big_poe_riding branch June 9, 2025 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component: ASM/C Changes some internals of the ASM/C libraries Racing Impact Changes a mechanic in a way that impacts the balance of competitive racing. Type: Enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants