Skip to content

Fix boss title card extraction for NTSC; Fix Morpha water#5320

Merged
leggettc18 merged 1 commit intoHarbourMasters:develop-blairfrom
Archez:fix-ntcs-bosstitle-extract
Apr 4, 2025
Merged

Fix boss title card extraction for NTSC; Fix Morpha water#5320
leggettc18 merged 1 commit intoHarbourMasters:develop-blairfrom
Archez:fix-ntcs-bosstitle-extract

Conversation

@Archez
Copy link
Contributor

@Archez Archez commented Apr 4, 2025

This updates the size of the boss title cards being extracted for NTCS. They only have two languages so they should be smaller in height than PAL. In some cases, this over extraction caused issues with auto Vtx extraction, which broken the DL for the water in Morpha's room.

Fixes #5308

Build Artifacts

@Archez Archez added this to the 9.0.1 milestone Apr 4, 2025
@leggettc18
Copy link
Contributor

Two Questions:

  1. Does this change require an OTR Regen? I'm guessing so.
  2. Will this cause the user to get prompted to regen their OTRs? I'm unfamiliar with how that code works.

@Archez
Copy link
Contributor Author

Archez commented Apr 4, 2025

Would require a regen. Will not crash with older OTR, just the same behavior will exist (invisible water).
Will not auto prompt for this.

In the past we handled this (xml changes in patch release) by making a callout in the changelog that a regen will be required for specific fixes.

@leggettc18
Copy link
Contributor

Works for me, was just curious about the OTR situation. For the record, I tested a version before this patch and this patch, and this did fix Morpha's water but the titlecard looks the same on both. Is this one of those situations where the incorrect size didn't visually mess anything up or was Morpha's titlecard just unaffected?

@leggettc18
Copy link
Contributor

Oh wait I understand it now, the titlecard extraction being wrong is why the water was broken, nevermind.

@Archez
Copy link
Contributor Author

Archez commented Apr 4, 2025

That's because we additionally extract the title cards separately as split up textures (one for each language). So the ones I'm adjusting are technically unused in SoH because of our design changes.

@leggettc18 leggettc18 merged commit 2589448 into HarbourMasters:develop-blair Apr 4, 2025
6 checks passed
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.

2 participants