Skip to content

[PoC] Rando "useless" checks & Potential for custom actor spawning/replacement system - #1176

Closed
garrettjoecox wants to merge 1 commit into
HarbourMasters:rando-nextfrom
garrettjoecox:useless-checks
Closed

[PoC] Rando "useless" checks & Potential for custom actor spawning/replacement system#1176
garrettjoecox wants to merge 1 commit into
HarbourMasters:rando-nextfrom
garrettjoecox:useless-checks

Conversation

@garrettjoecox

@garrettjoecox garrettjoecox commented Aug 13, 2022

Copy link
Copy Markdown
Contributor

Mostly putting this PR up to gauge interest on either topics

Useless Checks

This adds 5 checks to seemingly useless places, see images below

  • Gerudo Valley Octorok Grotto
  • Platform above bean patch in Kokiri Forest
  • Upper Platform near lizalfos fight
  • Frozen Pot in Ice Cavern Map room
  • Bottom room in Jabu Jabu's belly that just has pots

There is potential for more, but wanted to see if this is even desirable first.

Custom actor spawning/replacement system

The way this is achieved is hooking into the scene load commands and either appending our own actors or replacing existing actors with our own. The code in soh/soh/z_scene_otr.cpp accomplishes this but is very much proof of concept, if this is something we want to move forward with I'm thinking it will look something like @leggettc18 's custom item system where randomizer & other mods can hook in and provide actors/replacements.


Again this PR is not really ready to be merged, and will probably not be unless people want it. In case they do todo list for me:

  • Increase CheckTable/Spoiler table sizes
  • Create and migrate to a more generalized/flexible actor add/replacement system
  • Add randomizer setting for "Useless checks"
  • Maybe come up with a better name than useless checks

Screen Shot 2022-08-13 at 12 47 24 AM

Screen Shot 2022-08-13 at 12 50 16 AM

Screen Shot 2022-08-13 at 12 35 31 AM

Screen Shot 2022-08-13 at 12 51 34 AM

Screen Shot 2022-08-13 at 12 49 17 AM

@Alenmalen

Copy link
Copy Markdown

I love this idea very much, I had the same thoughts because there is a lot of places with potential for checks and those are some missed opportunities. Unfortunately my programming skills are basicly 0 but maybe I can provide you with some other checks that would be cool to add, if you´d like.

I especially like the idea of the check at the lizzard fights in DC because some parts of the game that are really cool are always skipped because there are no checks in that area, like this one for instance.
Also maybe putting 1 in the secret room behind the lava wall on the first flor of the lava room would be a nice idea, in Vanilla there is already a piece of heart inside, so maybe the actor can be used to make programming easier.

Anyway i support the idea of having an "Additional Checks" or something of that sort as an option for Rando, would really spice things up.
Thanks for putting in effort to making this project amazing.

@leggettc18

Copy link
Copy Markdown
Contributor

So, food for thought. I'm not 100% sure, but I think these all become "useful checks" in potsanity, which is planned to happen eventually. Similarly, generalized actor replacement/addition is also probably going to happen as part of the eventual modding engine. So you may have an opportunity here for contribution to that. I'm not sure what the plans are for that exactly.

@Kenix3
Kenix3 deleted the branch HarbourMasters:rando-next September 30, 2022 02:55
@Kenix3 Kenix3 closed this Sep 30, 2022
@garrettjoecox garrettjoecox self-assigned this Oct 1, 2022
serprex pushed a commit that referenced this pull request Jul 30, 2026
Advances the libultraship pin to the port-maintenance tip
(c57da1b -> bbb565b), resolving the HD texture-pack regressions
tracked in #6837:

- Kenix3/libultraship#1135 - stop clamping HD textures to an
  undefined tile region (fixes the eye rendering)
- Kenix3/libultraship#1172 - reshape RAW block-load texture uploads
  that arrive as a single line (fixes the C-button icons)

The related #1143 buttons/Metal fix was already in the prior pin.
Also in range: #1139, #1170, #1173, #1175, #1176.

Fixes #6837.

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants