Archipelago
Archipelago copied to clipboard
ArchipelagoMW
Muse Dash: Add filler items and rework generation balance
What is this fixing or adding?
This PR aims to fix a problem within Muse Dash. The total lack of filler items. This ends up changing the balance of the Muse Dash rando so other changes have been made.
Full list of changes:
- Added 3 filler items. These are 1 time consumables to Muse Dash.
- Made all songs have 2 locations. (Rather than a percentage of them)
- This was to simplify balance. (Having a variable amount of songs meant that Muse Dash could be generated without filler.)
- When filling the multiworld the following changes were made:
- Music Sheet count has been slightly lowered. (No Longer double counts starting songs.)
- After adding music sheets and the initial lot of songs (and traps), 50% of the remaining spots are now filled with filler.
- The final remaining slots are filled with duplicate songs. These songs are no longer progressive.
How was this tested?
Using multiple Beta Test games using discord. As well as existing tests to ensure no old functionality was broken.
That and i'm assuming the Unit test failure is a random failure, nothing failed on my machine (but I'm not running the same profile that failed)
Failure is from MMBN3 and is unrelated to the PR (for those concerned with the failure)
Thanks for catching the old options usage. And yes the old test failure was unrelated to the PR.
It should be impossible for the create items func to create more items than locations. It does seem to not care at first, but that is because all worlds will have enough space for a copy of every song, plus all the music sheets required. (As every song has 2 locations now.) And then every step after that checks to ensure it doesn't add too much.
Resolved the merge conflict. Bringing this to core-review.
Special note if merging before the big async. This does end up removing the option additional_item_percentage which doesn't affect too much, but might need a bit of manual adjusting of yamls.