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

Added blitz code for a faster playthrough. #7

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

phku-hk
Copy link

@phku-hk phku-hk commented Jul 30, 2023

No description provided.

lisa arrowlike
marie breakable
clare party1
daos_shrine lisa&marie&clare&any_key
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is not possible to fly to Doom Island without the engine; you should either add the engine back as a requirement to reach Daos Shrine, or create a modified event that does not require the engine.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assumed that was just a cutscene. sorry, my bad. I will try to create a modified event.

kamirno airship&party3
lisa arrowlike
marie breakable
clare party1
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure what is achieved by creating "lisa", "marie", and "clare" as locations. I don't know what your intent is for these lines, but I suspect that a required item might end up at one of these locations, which will make the seed unbeatable. For example, what if the bombs end up at the "lisa" location, which are then required to unlock the rest of the seed?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assumed that file defines routing requirements, so I added those maides. Also somewhere in your code* it tests the seed, so I did not assume any problems there. Also several people tested these requirements and had no problems. I will try to find such a "dead lock" you mentioned.

*itemrouter.py:703: success = self.try_unlock_locations(self.goal_requirements)

@phku-hk
Copy link
Author

phku-hk commented Jul 31, 2023

I fixed not being able to fly to doom island and tested it using a quick seed. This only affects blitz mode. You still need engine in non-blitz modes.
Also I changed blitz mode routing requirements to make it more interesting.

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