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

Updates auto-transfer configuration #11727

Merged
merged 1 commit into from
Oct 19, 2024

Conversation

Rukofamicom
Copy link
Contributor

About The Pull Request

Alright, it's been a little while with the initial default values and it's become clear that rounds are still mostly ending in chaos and destruction. The majority of players are still not actually taking part in the vote even when absolute shit has hit the fan and half the station is dead.

These config changes effective shave 30 minutes off of the old configuration from the front, and also result in more frequent reminders to vote.

Why It's Good For The Game

The average round length should be about two hours without forcing a two hour limit. These configuration settings have already been applied to the game to see how they affected things and have been shown to produce the desired results already - this PR (like all config PRs) exist primarily to announce the change to other players since config files are not actually changed on the server by PRs.

Testing Photographs and Procedure

Here is a screenshot of the current round with these changes already live:
image

Changelog

🆑
config: The percentage of votes required to leave now starts at 70% instead of 75%
config: Auto-transfer reminders to vote now happen every 40 minutes, and decay of the required votes to leave now begins at 40 minutes instead of 60.
config: These changes mean the maximum round length before a shuttle is forcibly called is now 3 hours instead of 3.5.
/:cl:

Copy link
Member

@PowerfulBacon PowerfulBacon left a comment

Choose a reason for hiding this comment

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

This is already applied and is the new recommended based on testing.

@PowerfulBacon PowerfulBacon added this pull request to the merge queue Oct 19, 2024
Merged via the queue into BeeStation:master with commit 6e8e1cc Oct 19, 2024
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants