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

Doubles the time you can get the "Long shift" achievement, makes it not grant on admin restarts #77195

Merged
merged 2 commits into from
Jul 29, 2023

Conversation

MrMelbert
Copy link
Contributor

@MrMelbert MrMelbert commented Jul 28, 2023

About The Pull Request

  • "Long shift" can now be earned from sub 10 minute rounds rather than sub 5 minute rounds

  • Admin restarts no longer give out "Long shift"

Why It's Good For The Game

I do not think this achievement can possibly be earned right now. Like at all.

Nuke Ops and cult are the only antags that can possibly do it and it's incredibly infeasible (requiring that they nuke the station or summon Nar'sie in just 3 minutes!)

So I bumped up the timer to 10 minutes. This means that ops can get it if they nuke the station in 8 minutes, cult can get it if they REALLY speedrun, and revs can get it if they beeline the heads.

I checked the DB for stats on this achievement and it's only been earned in 3 rounds across the last year - 208780 (admin restart due to a bug) 192892 (admin restart due to a bug?) 186192 (admin restart).

So I also prevented admin forcing the round to end. (I don't know if it catches admin reboots directly I'll have to check that.)

Changelog

🆑 Melbert
balance: The "Long Shift" achievement is now feasibly obtainable, and admins can no longer trigger it unknowingly
/:cl:

@tgstation-server tgstation-server added the Balance Changes to functionality that modifies how effective certain methods are at powergaming label Jul 28, 2023
@san7890 san7890 merged commit 298b5d3 into tgstation:master Jul 29, 2023
github-actions bot added a commit that referenced this pull request Jul 29, 2023
comfyorange added a commit that referenced this pull request Jul 29, 2023
NaakaKo referenced this pull request in Bird-Lounge/Skyraptor-SS13 Jul 29, 2023
…ot grant on admin restarts (#77195)

## About The Pull Request

- "Long shift" can now be earned from sub 10 minute rounds rather than
sub 5 minute rounds

- Admin restarts no longer give out "Long shift"

## Why It's Good For The Game

I do not think this achievement can *possibly* be earned right now. Like
at all.

Nuke Ops and cult are the only antags that can possibly do it and it's
incredibly infeasible (requiring that they nuke the station or summon
Nar'sie in just 3 minutes!)

So I bumped up the timer to 10 minutes. This means that ops can get it
if they nuke the station in 8 minutes, cult can get it if they REALLY
speedrun, and revs can get it if they beeline the heads.

I checked the DB for stats on this achievement and it's only been earned
in 3 rounds across the last year - `208780` (admin restart due to a bug)
`192892` (admin restart due to a bug?) `186192` (admin restart).

So I also prevented admin forcing the round to end. (I don't know if it
catches admin reboots directly I'll have to check that.)

## Changelog

:cl: Melbert
balance: The "Long Shift" achievement is now feasibly obtainable, and
admins can no longer trigger it unknowingly
/:cl:
Jolly-66 pushed a commit to TaleStation/TaleStation that referenced this pull request Jul 29, 2023
…chievement, makes it not grant on admin restarts (#7046)

Original PR: tgstation/tgstation#77195
-----
## About The Pull Request

- "Long shift" can now be earned from sub 10 minute rounds rather than
sub 5 minute rounds

- Admin restarts no longer give out "Long shift"

## Why It's Good For The Game

I do not think this achievement can *possibly* be earned right now. Like
at all.

Nuke Ops and cult are the only antags that can possibly do it and it's
incredibly infeasible (requiring that they nuke the station or summon
Nar'sie in just 3 minutes!)

So I bumped up the timer to 10 minutes. This means that ops can get it
if they nuke the station in 8 minutes, cult can get it if they REALLY
speedrun, and revs can get it if they beeline the heads.

I checked the DB for stats on this achievement and it's only been earned
in 3 rounds across the last year - `208780` (admin restart due to a bug)
`192892` (admin restart due to a bug?) `186192` (admin restart).

So I also prevented admin forcing the round to end. (I don't know if it
catches admin reboots directly I'll have to check that.)

## Changelog

:cl: Melbert
balance: The "Long Shift" achievement is now feasibly obtainable, and
admins can no longer trigger it unknowingly
/:cl:

---------

Co-authored-by: MrMelbert <51863163+MrMelbert@users.noreply.github.com>
@MrMelbert MrMelbert deleted the speed_round_tweak branch July 29, 2023 21:02
Gboster-0 referenced this pull request in Gboster-0/Monkestation2.0 Nov 12, 2023
…ot grant on admin restarts (#77195)

- "Long shift" can now be earned from sub 10 minute rounds rather than
sub 5 minute rounds

- Admin restarts no longer give out "Long shift"

I do not think this achievement can *possibly* be earned right now. Like
at all.

Nuke Ops and cult are the only antags that can possibly do it and it's
incredibly infeasible (requiring that they nuke the station or summon
Nar'sie in just 3 minutes!)

So I bumped up the timer to 10 minutes. This means that ops can get it
if they nuke the station in 8 minutes, cult can get it if they REALLY
speedrun, and revs can get it if they beeline the heads.

I checked the DB for stats on this achievement and it's only been earned
in 3 rounds across the last year - `208780` (admin restart due to a bug)
`192892` (admin restart due to a bug?) `186192` (admin restart).

So I also prevented admin forcing the round to end. (I don't know if it
catches admin reboots directly I'll have to check that.)

:cl: Melbert
balance: The "Long Shift" achievement is now feasibly obtainable, and
admins can no longer trigger it unknowingly
/:cl:
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance Changes to functionality that modifies how effective certain methods are at powergaming
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants