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

Wave Pirates fix and QoL tweaks #22

Merged
merged 2 commits into from
Mar 8, 2024
Merged

Wave Pirates fix and QoL tweaks #22

merged 2 commits into from
Mar 8, 2024

Conversation

MrMiguel211
Copy link

@MrMiguel211 MrMiguel211 commented Feb 13, 2024

Skippable Cutscenes:

  • Fixed an issue in Mine Security Station where the wave pirates won't drop down if the player didn't skip the cutscene immediately.

QoL:

  • Increased the size of the top and bottom door open triggers in Root Cave
  • Mine Security Station:
    • Changed the Shadow Pirates to activate the Wave Pirates cutscene upon DeathRattle instead of Dead
    • Resized the cutscene trigger so it's not accidentaly skipped if jumped over.

@Uncle-Reggie
Copy link

I’m not sure I agree with the changes to vanilla behavior in MSS. There are still people who get Storage Depot A and use the cutscene to repo back to where the Wave Pirates are in MSS. I am one of them.

@Uncle-Reggie
Copy link

Both triggers are good ideas.

@toasterparty
Copy link

what reggie said

@duncathan
Copy link

if the qol changes aren't strictly necessary for the fix to work, they should be in a separate PR probably

@MrMiguel211
Copy link
Author

MrMiguel211 commented Feb 17, 2024

I’m not sure I agree with the changes to vanilla behavior in MSS. There are still people who get Storage Depot A and use the cutscene to repo back to where the Wave Pirates are in MSS. I am one of them.

Counterpoints:

  1. If you kill the Wave Pirates first then the Shadow Pirates, it would be pretty confusing to see an empty cutscene that replays the music and everything else except that nothing happens.

  2. Some people would find it really annoying to be on their way out of the room and kill the shadow pirates for convenience only to be repositioned to the top of the room and need to walk down again.

  3. I doubt many other people would care to do speedrun strats in a randomizer but this is just my opinion.

I can still veto this if desired.

@Uncle-Reggie
Copy link

I’m not sure I agree with the changes to vanilla behavior in MSS. There are still people who get Storage Depot A and use the cutscene to repo back to where the Wave Pirates are in MSS. I am one of them.

Counterpoints:

  1. If you kill the Wave Pirates first then the Shadow Pirates, it would be pretty confusing to see an empty cutscene that replays the music and everything else except that nothing happens.
  2. Some people would find it really annoying to be on their way out of the room and kill the shadow pirates for convenience only to be repositioned to the top of the room and need to walk down again.
  3. I doubt many other people would care to do speedrun strats in a randomizer but this is just my opinion.

I can still veto this if desired.

Toaster and I had come up with a good grade for the “cost” of a QoL being introduced, and the other MSS changes fall into the “lose more than you gain” category for me.

The behavior is expected.

@Miepee
Copy link

Miepee commented Feb 17, 2024

From a casual perspective, these changes are nice, but it'd probably be better to have them seperate from QoL. So that speedrunners who have the muscle memory burned down to use reposition strats can continue to rely on them and newcomers being less confused.

@Samuel6710
Copy link

If you kill the Wave Pirates first then the Shadow Pirates, it would be pretty confusing to see an empty cutscene that replays the music and everything else except that nothing happens.

Some people would find it really annoying to be on their way out of the room and kill the shadow pirates for convenience only to be repositioned to the top of the room and need to walk down again.

Are these really valid issues? I genuinely do not see any complaints regarding this vanilla behavior, and many have already been playing with this behavior implemented so they have come to expect it, so reverting this would be counterintuitive because players already expect vanilla behavior out of this. If players understand this behavior fully, they can learn to take advantage of the reposition and make some pretty creative plays in the same way a speedrunner does.

@toasterparty toasterparty self-requested a review February 18, 2024 21:31
Copy link

@toasterparty toasterparty left a comment

Choose a reason for hiding this comment

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

Removed vanilla behavior of the cutscene playing again upon killing the Shadow Pirates if entering the room in reverse.

Retract this change. Everything else is good

@MrMiguel211
Copy link
Author

Retract this change. Everything else is good

I'm still working on some other commits, will probably finish today.

@toasterparty
Copy link

Are those commits going into this branch? The less you dogpile into one PR, the faster it is for me to review/approve/merge

@MrMiguel211
Copy link
Author

Are those commits going into this branch? The less you dogpile into one PR, the faster it is for me to review/approve/merge

mmm I suppose I'll make a different PR then, I still have to change the thing tho

@toasterparty toasterparty merged commit 9507710 into randovania:main Mar 8, 2024
3 checks passed
@MrMiguel211 MrMiguel211 deleted the QoL-fixes-and-tweaks branch March 13, 2024 02:54
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.

6 participants