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

removes detective's roundstart lethals #4059

Closed
wants to merge 1 commit into from
Closed

removes detective's roundstart lethals #4059

wants to merge 1 commit into from

Conversation

mopcat
Copy link
Contributor

@mopcat mopcat commented Mar 18, 2021

[BALANCE][FEEDBACK]

About the PR

removes lethals from det's revolver box, adds an extra nonlethal speedloader to compensate

Why's this needed?

too many stinky dets using lethals when lethals are not necessary, which i feel is compounded by the fact that they literally spawn with lethal rounds in their backpack. if the det really wants to kill people, they can either put in the effort required to have lethal bullets by hacking a secvend, which will get a few raised eyebrows from real sec, or begging the head of security for the ammotech speedloaders. or they can do it like the rest of us (with blunt objects). hopefully reduces pain caused by stinko detectives

detective is also not meant to be making arrests or killing people in the first place, they're meant to be a forensics role. the gun is there for self defense, and using lethals in self defense and having that behavior encouraged by spawning them with lethals is not good.

Changelog

(u)mopcat
(+)removes lethals from det's revolver box, adds an extra nonlethal speedloader to compensate

@keywordlabeler keywordlabeler bot added the C-Balance Balance changes, buffs and nerfs label Mar 18, 2021
@mopcat mopcat changed the title please fucking work fml removes detective's roundstart lethals Mar 18, 2021
@Kyle2143
Copy link
Contributor

I think that this is a non-issue or just a reactionary change. Some people play detective badly. If they kill someone with lethal ammunition without warrant, they should be ahelped because they are breaking the rules. Removing their lethal ammunition, leaving them with only stuns without hacking or opening the armory is like punishing every decent detective player for their rulebreaking.

@mat-lap
Copy link

mat-lap commented Mar 18, 2021

I think that this is a non-issue or just a reactionary change. Some people play detective badly. If they kill someone with lethal ammunition without warrant, they should be ahelped because they are breaking the rules. Removing their lethal ammunition, leaving them with only stuns without hacking or opening the armory is like punishing every decent detective player for their rulebreaking.

this tbh. i have rarely seen detectives abusing lethals, and those who do get ahelped. it's a non-issue

@mopcat
Copy link
Contributor Author

mopcat commented Mar 18, 2021

I think that this is a non-issue or just a reactionary change. Some people play detective badly. If they kill someone with lethal ammunition without warrant, they should be ahelped because they are breaking the rules. Removing their lethal ammunition, leaving them with only stuns without hacking or opening the armory is like punishing every decent detective player for their rulebreaking.

the reason this change is being made is because the detective shouldn't have lethals to begin with imo. security and all roles adjacent to it should be using nonlethal force unless absolutely necessary, and the detective just starting with lethals from the moment they spawn goes directly against that. plus reducing the amount of lethals they have access to would reduce how often they're abused.

and yes, i made this change after seeing three separate det players in a row on rp gun people down with lethals despite having several stunner rounds still on them. i'd rather make an in-game solution to this than have to adminhelp almost every detective player who uses the lethals they start with in the first place

@mopcat
Copy link
Contributor Author

mopcat commented Mar 18, 2021

i also want to mention that a controversial change effectively punishing everyone was made just a few months ago to ntso, which effectively punished all ntso players for the actions of a few bad eggs, which would've been better fixed imo by enforcing the rules on those people and officially clarifying what that role was meant to be.

nothing against anyone in particular here, just pointing this out because it's really making me think

@keywordlabeler keywordlabeler bot added the E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums. label Mar 18, 2021
@github-actions
Copy link
Contributor

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

Copy link
Contributor

@geneva-breaker geneva-breaker left a comment

Choose a reason for hiding this comment

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

perhaps it is best if simply we do not do these things, and instead do the original thing which works fine when people follow the rules and use A-help when they don't.

@@ -60,11 +60,10 @@
/obj/item/storage/box/detectivegun
name = ".38 revolver box"
icon_state = "hard_case"
desc = "A box containing a .38 caliber revolver and ammunition."
desc = "A box containing a .38 caliber revolver and non-lethal ammunition."
Copy link
Contributor

Choose a reason for hiding this comment

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

suggestion: desc = "A box containing a .38 caliber revolver and ammunition."

// Reduced the amount of ammo. The detective had four lethal and five stun speedloaders total in his closet, perhaps a bit too much (Convair880).
spawn_contents = list(/obj/item/gun/kinetic/detectiverevolver,\
/obj/item/ammo/bullets/a38 = 2,\
/obj/item/ammo/bullets/a38/stun = 2)
/obj/item/ammo/bullets/a38/stun = 3)
Copy link
Contributor

Choose a reason for hiding this comment

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

suggestion:
/obj/item/ammo/bullets/a38 = 2,\ /obj/item/ammo/bullets/a38/stun = 2)

@Camryn-Buttes
Copy link
Contributor

I'm with kyle on this.

This feels like a reactionary change addressing something that isn't actually a problem.

I personally also disagree with this:

security and all roles adjacent to it should be using nonlethal force unless absolutely necessary

Hard and strong disagree.

If you had made this pr exclusively for the rp servers, I might be inclined to agree, but it's just incompatible with main.

@mopcat mopcat closed this Mar 18, 2021
@mopcat
Copy link
Contributor Author

mopcat commented Mar 18, 2021

remind me never to try contributing for this game again ugh

@mopcat mopcat deleted the fuck-off-murderdet branch March 18, 2021 04:04
@flappybatpal
Copy link
Contributor

Mop don't overreact. Changing existing features and adding new ones are not easy as people have opinions. This is a community project, you have to navigate opinions and find a way forward. Maybe that means the change will never happen, much bigger pieces of work have been turned down.

Start with Discord discussion and a forum thread to feel out the general opinion unless you are comfortable with things not making it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Balance Balance changes, buffs and nerfs E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants