-
Notifications
You must be signed in to change notification settings - Fork 776
Feature/split songs #6627
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
Open
RaccoonCloud
wants to merge
25
commits into
HarbourMasters:develop
Choose a base branch
from
RaccoonCloud:feature/split-songs
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Feature/split songs #6627
Changes from 24 commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
9397b10
iAdd split ocarina songs mode with part-based progression.
RaccoonCloud 5feddec
iUpdate split songs file headers with creator attribution.
RaccoonCloud 2bd2ea4
Merge develop into feature/split-songs and resolve conflicts
RaccoonCloud 5417774
Apply clang-format to split songs changes
RaccoonCloud 3d0a0f6
RaccoonCloud 0386051
Update split songs option text for progressive behavior.
RaccoonCloud 6d49e4d
Randomizer: pickup textbox icons and progressive split songs
RaccoonCloud 042e035
Fix clang-format in ItemMessages.cpp
RaccoonCloud 6a10537
Randomizer: all song textbox icons work for progressive pickup
RaccoonCloud bc2027e
Simplify split songs to progressive-only RandInf tracking
RaccoonCloud 72c849f
Address split songs PR review feedback
RaccoonCloud bd13ce5
Merge upstream develop into feature/split-songs
RaccoonCloud 11a4287
Fix ItemMessages build after develop merge
RaccoonCloud f8464be
Bump libultraship to match upstream develop after merge
RaccoonCloud 10f462d
Fix split song tracker texture lookup for Fast3dGui API
RaccoonCloud 2340e5a
Fix seed generation crash when regenerating with split songs
RaccoonCloud 3e1aff7
Fix split song part-1 grant and restore key ring icons
RaccoonCloud 2998034
Simplify split songs to RandInf + progressive-only pattern.
RaccoonCloud 600b8d3
Address split songs review: settings, pool, tracker, traps.
RaccoonCloud 39427cc
Fix seed gen crash on progressive songs and clean up split songs menu…
RaccoonCloud b78aac9
Fix split song pickup icons and stabilize seed/save generation
RaccoonCloud 79b6882
Fix Linux CI: use SaveData string overload for UTF-8 sanitization.
RaccoonCloud 1198703
Merge upstream develop into feature/split-songs
RaccoonCloud 7d146a6
Split PR scope per review: drop icon and UTF-8 side fixes.
RaccoonCloud 2edee8d
Refactor split songs to PART items and static songData maps per review.
RaccoonCloud File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you explain why you added these?
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you again for your help and patience with me (first timer — I kept messaging Cal).
Every time I ran a new seed or generated one, SoH was crashing because other things (item tracker, spoiler lookups) could read placement data while Fill() was still mid-reset — like a “seed ready” race.
I SetSeedGenerated(false) at the top of Playthrough_Init and only flip to true after SpoilerLog_Write() finishes, so nothing treats the seed as valid until the spoiler JSON is fully written. Before that, things were trying to use spoiler data that wasn’t ready yet and it’d just crash on generate. If there’s a better way I’ll relook and sort.
This showed up most with split songs because the tracker hits song progress during seed generation, but the flag lifecycle fix is general — it just enforces generate → write → consume order.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rebuilt and tested split songs still works (first pickup = part / 1/2 on tracker, second = full song).
Pulled icon hooks and UTF-8 stuff into separate branches like you asked.
Also removed the logic scratch workaround progressive songs go through normal ApplyItemEffect now