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

Cuffs rebalance #13052

Closed
wants to merge 3 commits into from
Closed

Conversation

CheffieGithub
Copy link
Contributor

@CheffieGithub CheffieGithub commented Feb 11, 2023

About the PR

Shakes up handcuffs a lil bit

Removal times
normal cuffs: 40 - 50 seconds to 25 - 35 seconds
custom material cuff: max cuff time 50 seconds to 40 seconds

Apply times
Overall 6 seconds
Security bonus: 2x to 1.5x
ziptie cuffs 0.5x apply
normal cuffs: 6 seconds crew / 4 seconds security
ziptie cuffs: 3 seconds crew / 2 seconds security

before was 4 seconds crew / 2 security for both

Pouch changes
Security officers now spawns with 2 pairs of ziptie cuffs and 1 pair normal
Security assistants now spawn with 2 pairs of ziptie cuffs

Why's this needed?

I think that currently cuffs are too strong. If you do get cuffed they are either a non issue due to strength or you need to stand still for almost a full minute to remove them, its just too punishing.
There is also no reason at all to use ziptie cuffs since they take the same time to cuff someone. Now ziptie cuffs should be the standard for most arrests with normal cuffs being reserved for higher priority targets with the slightly longer cuff time.

Changelog

(u)Cheffie
(+)Cuffs have been rebalanced slightly:
Normal cuffs now take a bit longer to apply and less time to remove. Ziptie cuffs take less time to apply. 
Security now spawns with some ziptie cuffs in their pouches.

@github-actions github-actions bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Feb 11, 2023
@CheffieGithub CheffieGithub added A-Game-Objects The point of this PR is to deal with a specific game object C-Balance Balance changes, buffs and nerfs labels Feb 11, 2023
@MertyMor
Copy link
Contributor

not quite sure about this one, think a forum post might be good to discuss it

@DisturbHerb DisturbHerb added the E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums. label Feb 11, 2023
@github-actions
Copy link
Contributor

Created discussion thread: https://forum.ss13.co/showthread.php?tid=20689

@Tarmunora
Copy link
Member

Not really a fan of these changes, at least not all together. This PR ends up being a large shotgun nerf to handcuffs by hitting availability, apply time, and cuff duration all at once.

@CheffieGithub
Copy link
Contributor Author

You have to consider the fact that security still has a massive supply of handcuffs, the majority of stun times will still allow for cuffs with a gracious amount of time spare and even with ziptie cuffs there will be more than enough time to search someone even more so if they are buckled.

Maybe this PR is a good candicate for a test merge? won't know how it plays otherwise.

@colossusqw
Copy link
Contributor

You have to consider the fact that security still has a massive supply of handcuffs, the majority of stun times will still allow for cuffs with a gracious amount of time spare and even with ziptie cuffs there will be more than enough time to search someone even more so if they are buckled.

Maybe this PR is a good candicate for a test merge? won't know how it plays otherwise.

This seems like a very big and overarching change, but i would personally love to see how it goes with a test merge. Could end up horribly, but it could also just work and giving it a couple of days wouldn't really be horrible even if it turns out to be the wrong call...

@flappybatpal
Copy link
Contributor

Also have my doubts about these changes.

@github-actions
Copy link
Contributor

This PR has been inactive for two weeks, and has been automatically marked as stale. This means it is at risk of being auto closed in another week. Please address any outstanding review items and ensure your PR is finished. If you are auto-staled anyway, ask developers if your PR will be merged. Once you have done any of the previous actions then you should request a developer remove the stale label on your PR, to reset the stale timer. If you feel no developer will respond in that time, you may wish to close this PR youself, while you seek developer comment, as you will then be able to reopen the PR yourself.

@github-actions github-actions bot added the S-Stale An inactive PR that has had no updates in the past two weeks label Mar 15, 2023
@github-actions github-actions bot closed this Mar 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Game-Objects The point of this PR is to deal with a specific game object C-Balance Balance changes, buffs and nerfs E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums. S-Stale An inactive PR that has had no updates in the past two weeks size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants