randovania / randomprime

Prime1 patcher for Randovania and a few other things.
MIT License
2 stars 3 forks source link

Wave Pirates fix and QoL tweaks #22

Closed MrMiguel211 closed 8 months ago

MrMiguel211 commented 9 months ago

Skippable Cutscenes:

QoL:

Uncle-Reggie commented 8 months ago

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 commented 8 months ago

Both triggers are good ideas.

toasterparty commented 8 months ago

what reggie said

duncathan commented 8 months ago

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

MrMiguel211 commented 8 months ago

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 commented 8 months ago

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 commented 8 months ago

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 commented 8 months ago

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.

MrMiguel211 commented 8 months ago

Retract this change. Everything else is good

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

toasterparty commented 8 months ago

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 commented 8 months ago

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