Skip to content
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

DLC Quest Stardew bug #2423

Merged
merged 2 commits into from
Nov 6, 2023
Merged

Conversation

axe-y
Copy link
Collaborator

@axe-y axe-y commented Nov 4, 2023

I fix a bug where I made an item early but it didn't exist so stardew was waiting for the early item to be collected and the generation exploded.
I also finally added get_filler_item_name

ex. "MyGame4: implement new game" or "Docs: add new guide for customizing MyGame3"

What is this fixing or adding?

a generation bug between DLC Quest and Stardew

How was this tested?

ran the yaml that didn't worked before with success

If this makes graphical changes, please attach screenshots.

@el-u
Copy link
Collaborator

el-u commented Nov 4, 2023

So we've reached the point where other worlds have to work around Stardew Valley's rogue behavior...

@alwaysintreble
Copy link
Collaborator

while this fix works, I disagree with it on a fundamental level.

@ScipioWright ScipioWright added the is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs. label Nov 5, 2023
@Berserker66 Berserker66 merged commit 84fb2f5 into ArchipelagoMW:main Nov 6, 2023
12 checks passed
@Jouramie
Copy link
Contributor

Jouramie commented Nov 7, 2023

Even tho it's merged already, for a documentation purpose, I want to reinforce that there is no relation to Stardew with this bug (other than the detection of the issue). DLC Quest sets an item early without adding it to the pool, and this should be completely illegal (and tested). I would even expect the generation to always crash because of that. An early item is impossible to collect, the multi world should not even be solvable.

For Stardew, we had to work around a limitation in the core causing all event to be collected in the first sphere, which caused all seasons to be merged altogether; making some seeds literally unplayable. The solution of hooking into early items of other worlds might not be the most elegant (and does not always work if there is no early item in the multi world), but at least it gets the job done, in a creative manner.

@el-u
Copy link
Collaborator

el-u commented Nov 7, 2023

Hard disagree.

The only thing that should be illegal here is one world inspecting the parameters of other worlds.

FlySniper pushed a commit to FlySniper/Archipelago that referenced this pull request Nov 14, 2023
Jouramie pushed a commit to Jouramie/Archipelago that referenced this pull request Feb 28, 2024
@axe-y axe-y deleted the DLCQuest_Stardew_bug branch November 27, 2024 22:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants