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

Remove detain mode as a toggleable switch for end user #10282

Closed
wants to merge 2 commits into from

Conversation

optimumtact
Copy link
Member

Functionality is left in to support killbots, (so they don't try to handcuff you)

Note that killbots will not announce when engaging

This is still a problem with people getting permastunned, so I've reopened the PR removing it

Functionality is left in to support killbots, (so they don't try to handcuff you)

Note that killbots will not announce when engaging
@optimumtact optimumtact changed the title Remove detain mode as a toggleable switch for end user Remove detain mode as a toggleable switch for end user [DNM] Jun 30, 2015
@RemieRichards RemieRichards added the Do Not Merge You must have really upset someone label Jun 30, 2015
@AKlimentyev
Copy link

You could just add a delay to it, so the criminal can move a few tiles or try to hit them before being stunned again.
Detain is really useful in situations where cuffing someone isn't going to help as they can just run away.
Also beepsky on detain plus a hacked medibot is a good strategy for rogue Ais to assasinate someone.

@Cheridan
Copy link
Contributor

Yeah, Detain can lead to situations where the perp is just stunlocked for the rest of the round. It'd be ok if it had diminishing returns or if the bot would eventually wander off or something, but as it is it isn't really cool.

@optimumtact
Copy link
Member Author

Please follow the guidelines in the OP

@Fox-McCloud
Copy link
Contributor

If you're going to do this for ED209's, do it for Beepsky too.

@Dorsisdwarf
Copy link
Contributor

ED-209s lose it because they have a taser instead of a stunbaton, and more health.
You can outrun the long baton of the law, but not the golden bolt.

@Fox-McCloud
Copy link
Contributor

I think the point here is that the detain feature is pretty shitty because if you get caught (beepsky or ED209), you're pretty much permastunned, forever. Yes it's easier to deal with, with beepsters, but the feature it's is pretty shit.

@optimumtact
Copy link
Member Author

People are indicating you can run away from beepsky's detain stun, I'll have to test that out and see.

@Lumipharon
Copy link
Contributor

I haven't seen detain for a LONG ass time, but from what I remember, BEFORE the stun nerfs, detain mode would perma stun. But the stun durations got nerfed, but the delay between stuns for detain mode was not, so you could get up between stuns.

I could be wrong though.

@Fox-McCloud
Copy link
Contributor

It stuns you for 5 cycles (roughly ~10 seconds) and bots process every cycle (~2 seconds), so yeah, no way to escape once you're detained, AFAIK.

@Rinoka
Copy link
Contributor

Rinoka commented Jul 1, 2015

Why not just make it so that the bots have to recharge their taser/stunbaton and will do it automatically either when not busy or when they run out?

Means permastunning ain't no thing.

@Fox-McCloud
Copy link
Contributor

Because it's easier to remove a shit feature to begin with?

@Rinoka
Copy link
Contributor

Rinoka commented Jul 5, 2015

Point.

@optimumtact
Copy link
Member Author

if someone wants to take over this

@optimumtact optimumtact deleted the detainmyass branch September 16, 2015 22:56
@optimumtact optimumtact restored the detainmyass branch January 15, 2016 04:11
@optimumtact optimumtact reopened this Jan 15, 2016
@optimumtact optimumtact reopened this Jan 15, 2016
@optimumtact optimumtact changed the title Remove detain mode as a toggleable switch for end user [DNM] Remove detain mode as a toggleable switch for end user Jan 15, 2016
Conflicts:
	code/modules/mob/living/simple_animal/bot/ed209bot.dm
@Dorsisdwarf
Copy link
Contributor

Not ready for merge unless travis is lie

@GunHog
Copy link
Contributor

GunHog commented Jan 15, 2016

What? No, zombing a long dead PR to try and slip it past people as well as bypass the 24HR limit should be unacceptable. At that point, it needs to be treated as a new PR.

@optimumtact
Copy link
Member Author

your paranoia is outstanding

@tkdrg
Copy link
Contributor

tkdrg commented Jan 15, 2016

Stop trying to use necromancy to bypass the freeze

@tkdrg tkdrg closed this Jan 15, 2016
@Cheridan
Copy link
Contributor

He's not trying bypassing the freeze. Someone was talking about this topic and he reopened the old pr. Obviously don't merge it right away, treat it as a new pr

@Cheridan Cheridan reopened this Jan 16, 2016
@optimumtact optimumtact reopened this Jan 27, 2016
@optimumtact
Copy link
Member Author

treat this as a new PR so gunhog doesn't freak out

@GunHog
Copy link
Contributor

GunHog commented Jan 27, 2016

Still against it 👎

@AKlimentyev
Copy link

Still against it as well. 👎

@imtakingabreakdontatme
Copy link
Contributor

Back in my day beepsky didnt shout out the criminal location while holding them in place

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do Not Merge You must have really upset someone
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet