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

Removed bola stam damage #32989

Merged
merged 2 commits into from
Nov 10, 2024
Merged

Removed bola stam damage #32989

merged 2 commits into from
Nov 10, 2024

Conversation

ARMOKS
Copy link
Contributor

@ARMOKS ARMOKS commented Oct 24, 2024

Bola stam damage consistently allows for a 0.5 second stun, this can be done in 5 or 6 ways with different items, its bad for combat.

About the PR

Removed bola stam damage

Why / Balance

Bola + Baton / prod throw (wall bounce) = 0.5 second stun
Bola + Prod throw = 1 second stun
Bola + Boxing gloves = 2-3 second stun
Bola + Rigged gloves = ~1 second stun
There are several other combos that are absolutely disgusting, this on top of the fact that you can bind remove from backpack and throw to the same key to insta launch bolas without warning makes them a mess. A solution that would also be fine is giving them some sort of slowdown from taking from bag to throwing. IIRC they do not even have an in-hand sprite.

Changelog
🆑 remove bola stam damage

Bola stam damage consistently allows for a 0.5 second stun, this can be done in 5 or 6 ways with different items, its bad for combat.
@github-actions github-actions bot added the No C# For things that don't need code. label Oct 24, 2024
@ARMOKS
Copy link
Contributor Author

ARMOKS commented Oct 24, 2024

If anyone is concerned about this I can record a few of the bola interactions and post them here to show what I mean

@Futuristic-OK
Copy link
Contributor

Make stun bola for secfab, before makeshift bolas nerf.

@ARMOKS
Copy link
Contributor Author

ARMOKS commented Oct 25, 2024

Make stun bola for secfab, before makeshift bolas nerf.

half the issue is as sec I can bola someone then disabler or throw a baton at them for a near instant unstoppable stun

@ArtisticRoomba
Copy link
Contributor

a bola should restrain my movement as it wraps around my legs, I should not all of a sudden become physically exhausted and be one/two disabler shot(s) from collapsing onto the floor

this balance change is completely fine, IMO

@ninruB
Copy link

ninruB commented Oct 25, 2024

I think this essentially puts bolas out of commission like they were before they got stamina damage, but having 55 on-demand instant stamina damage on a makeshift item is ridiculous and shouldn't be a thing, especially considering that the stamina slow from taking 50 stamina damage will stack with whatever the bola itself applies.

Bolas should probably get a greater slowdown with this change.

@thebadman4662
Copy link

thebadman4662 commented Oct 25, 2024

You forgor:

Bola + DBS = ~0.2 second stun.

I think this essentially puts bolas out of commission like they were before they got stamina damage, but having 55 on-demand instant stamina damage on a makeshift item is ridiculous and shouldn't be a thing, especially considering that the stamina slow from taking 50 stamina damage will stack with whatever the bola itself applies.

Bolas should probably get a greater slowdown with this change.

AOE Flash into bola still results into chase end by sec without burning metric tons of flashses(wardens hate "wasting" 0.4 plastic per charge by printing basic supplies). It will also still be a good trapping/stalling weapon against nukies or zombies, even if bit worse.

@WhiteGorila
Copy link

Add Lazer bolas that cut people's legs off

@ARMOKS
Copy link
Contributor Author

ARMOKS commented Oct 25, 2024

I think this essentially puts bolas out of commission like they were before they got stamina damage, but having 55 on-demand instant stamina damage on a makeshift item is ridiculous and shouldn't be a thing, especially considering that the stamina slow from taking 50 stamina damage will stack with whatever the bola itself applies.

Bolas should probably get a greater slowdown with this change.

It absolutely does not, sec has a big issue with tiders running and slipping them and whatnot. Throwing a bola instantly ends this as they can no longer run and sec can VERY easily stun them. Even without stam damage a massive slowdown is a game changing thing in combat

@ARMOKS
Copy link
Contributor Author

ARMOKS commented Oct 25, 2024

Add Lazer bolas that cut people's legs off

I’ll PR this the second newmed drops

@thebadman4662
Copy link

thebadman4662 commented Oct 25, 2024

I think this essentially puts bolas out of commission like they were before they got stamina damage, but having 55 on-demand instant stamina damage on a makeshift item is ridiculous and shouldn't be a thing, especially considering that the stamina slow from taking 50 stamina damage will stack with whatever the bola itself applies.
Bolas should probably get a greater slowdown with this change.

It absolutely does not, sec has a big issue with tiders running and slipping them and whatnot. Throwing a bola instantly ends this as they can no longer run and sec can VERY easily stun them. Even without stam damage a massive slowdown is a game changing thing in combat

Bola is more of ambush/defense tool than chase one, throws are too slow to catch tiders unless there is already warrant on them for you to jump out of locker and throw it and they can't just weldmask it like flash/flashbang/clusterflash, provided there isn't someone, even dead in the way to block it when they stand still.

@SlamBamActionman
Copy link
Member

SlamBamActionman commented Oct 25, 2024

Instead of removing it, why not lower it to something like 20? That way you get some decent combo behavior with the stun baton/disabler without making the stun happen super fast.

Stamina slowdown happens at 50 stam damage. Stun baton deals 35 (2 hits = 70, 3 hits 105), Disabler deals 30 (2 hits = 60, 4 hits 120).

A bola throw at 20 means the first stun baton hit deals stamina slowdown, but you still need 3 baton hits to stun (20 > 55 > 90 > 125). This makes the bola + baton combo slower than just using the baton, but you still get a lot of benefit for using the bola.

For disabler, you also get a stun on the first hit and you get a stun after 3 hits instead of 4 (20 > 50 > 80 > 110), which feels appropriate for doing a more complex set of actions with a bola + 3 shots combo, compared to just shooting with the disabler 4 times.

@keronshb
Copy link
Contributor

Instead of removing it, why not lower it to something like 20? That way you get some decent combo behavior with the stun baton/disabler without making the stun happen super fast.

Stamina slowdown happens at 50 stam damage. Stun baton deals 35 (2 hits = 70, 3 hits 105), Disabler deals 30 (2 hits = 60, 4 hits 120).

A bola throw at 20 means the first stun baton hit deals stamina slowdown, but you still need 3 baton hits to stun (20 > 55 > 90 > 125). This makes the bola + baton combo slower than just using the baton, but you still get a lot of benefit for using the bola.

For disabler, you also get a stun on the first hit and you get a stun after 3 hits instead of 4 (20 > 50 > 80 > 110), which feels appropriate for doing a more complex set of actions with a bola + 3 shots combo, compared to just shooting with the disabler 4 times.

Alt to this -
Change the slowdown to mimic what it is now with the stam loss.

@Iamgoofball
Copy link

Iamgoofball commented Oct 26, 2024

Why don't you guys just make the bola work as originally designed; a flat slowdown until it's removed, no need to make it do any stamina damage. This is solved design. No need to reinvent the wheel here.

@keronshb
Copy link
Contributor

Why don't you guys just make the bola work as originally designed; a flat slowdown until it's removed, no need to make it do any stamina damage. This is solved design. No need to reinvent the wheel here.

That's exactly what I suggested.

@ARMOKS
Copy link
Contributor Author

ARMOKS commented Oct 27, 2024

Why don't you guys just make the bola work as originally designed; a flat slowdown until it's removed, no need to make it do any stamina damage. This is solved design. No need to reinvent the wheel here.

this is exactly what im doing, im gonna edit it a bit later because I realize I left out the makeshift bola

@WhiteGorila
Copy link

Why don't you guys just make the bola work as originally designed; a flat slowdown until it's removed, no need to make it do any stamina damage. This is solved design. No need to reinvent the wheel here.

Dude this is literally my response to 90 percent of the PRS on this game. This shit has already been done to death and figured out before literally all anyone has to do is copy from 13 it ain't that hard!!

@SlamBamActionman
Copy link
Member

Why don't you guys just make the bola work as originally designed; a flat slowdown until it's removed, no need to make it do any stamina damage. This is solved design. No need to reinvent the wheel here.

Dude this is literally my response to 90 percent of the PRS on this game. This shit has already been done to death and figured out before literally all anyone has to do is copy from 13 it ain't that hard!!

SS13 is a great resource to start from, but since SS14 is quite different when it comes to combat (with its movement especially!) it's definitely worth taking the time to evaluate changes in the context of SS14. Rote copying of SS13 without examining how it acts in SS14 risks introducing its own errors.

There's also a good chance that while SS13 has found something that works, there might be other stuff that works too! And in some cases those stuff might work even better!

And in some it might not. But if 90% of PRs aren't allowed to be exploratory, there's not gonna be much room for innovation, and I see no point in resting on SS13's laurels.

@Iamgoofball
Copy link

Iamgoofball commented Oct 28, 2024

I see no point in resting on SS13's laurels.

It's less about resting on SS13's laurels and more knowing when to recognize that something should be changed; a lot of SS13 design has been iterated on over the years via a lot of trial and error before it arrives at its current incarnations. When doing stuff like that, it's best to implement as originally designed, then tweak from there to account for how it plays on SS14 after a few weeks of testing, rather than starting with a modified design and then re-learning all the lessons we've learned over the multiple decades of development and playtesting at SS13.

There's definitely stuff to iterate on with pixel movement, but there's also a lot of spots where the iteration's already been done to death.

@deltanedas
Copy link
Contributor

being able to instantly no-counter stun someone with a macro or good keybinds is not a good thing regardless of ss13/ss14

@ARMOKS ARMOKS requested a review from keronshb November 9, 2024 19:03
@github-actions github-actions bot added the Status: Needs Review This PR requires new reviews before it can be merged. label Nov 9, 2024
Copy link
Contributor

@keronshb keronshb left a comment

Choose a reason for hiding this comment

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

lg2m

@keronshb keronshb merged commit d939e99 into space-wizards:master Nov 10, 2024
12 checks passed
@MilenVolf
Copy link
Contributor

CL was not set right

@TermoNemo

This comment was marked as off-topic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No C# For things that don't need code. Status: Needs Review This PR requires new reviews before it can be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.