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

Makes admins not count towards the playercount cap #33424

Merged
merged 3 commits into from
Dec 16, 2024

Conversation

deathride58
Copy link
Member

About the PR

This is a straight-forward PR, making it so that admins don't count towards the playercount cap. This is controlled by the new admin.admins_count_for_max_players cvar

Why / Balance

Currently, admins bypass the playercount cap entirely, meaning it's very common to see the servers end up escalating beyond their actual playercount cap. This means that when an admin joins while the server's at the cap, players have to wait for way longer than necessary for more than just one person to leave if they want to join the server, which is immensely frustrating for players. Simple fix: making it so that those who are able to bypass the playercount cap don't count towards it either.

Technical details

This adds a new CVar controlling whether admins count towards the playercount cap (if true, then it behaves exactly how the playercount cap behaved before).

When that CVar is false, the number of active admins gets subtracted from the current playercount during connection checks that care about the playercount.

Media

N/A

Requirements

Breaking changes

N/A

Changelog

🆑 Bhijn and Myr

  • tweak: Admins no longer count towards the playercount cap, meaning you no longer need to wait for multiple people to leave in order to join after an admin has joined.

@github-actions github-actions bot added size/S Denotes a PR that changes 10-99 lines. S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. labels Nov 20, 2024
@beck-thompson beck-thompson added P1: High Priority: Higher priority than other items, but isn't an emergency. T: New Feature Type: New feature or content, or extending existing content S: Needs Review Status: Requires additional reviews before being fully accepted A: Core Tech Area: Underlying core tech for the game and the Github repository. and removed S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. labels Nov 20, 2024
deathride58 and others added 2 commits November 19, 2024 22:26
Co-authored-by: Thomas <87614336+Aeshus@users.noreply.github.com>
Thanks VScode very good IDE
@SaphireLattice
Copy link
Contributor

This will still show "81/80" if there's two admins on, but will let one person join, if I read this right?

That has to be pretty confusing. It should remove them from the reported player count.

@SaphireLattice SaphireLattice added the S: Approved Status: Reviewed and approved by at least one maintainer; a PR may require another approval. label Nov 23, 2024
@ilovehans10
Copy link

This will still show "81/80" if there's two admins on, but will let one person join, if I read this right?

That has to be pretty confusing. It should remove them from the reported player count.

Honestly, it would be nice if there was an indicator for how many admins were online, and it could be separate from the player count still. Something where when you opened the dropdown for a server, it would have the server description, and then the number of admins online. That was this godsend can be implemented, and I can finally have a chance to hope on Harmony when there are 4 admins online (like I'm waiting for as I write this).\

Current:
Screenshot from 2024-11-23 13-29-29
Concept:
Admins online
(GIMP FTW)

@Aeshus
Copy link
Contributor

Aeshus commented Nov 23, 2024

This will still show "81/80" if there's two admins on, but will let one person join, if I read this right?
That has to be pretty confusing. It should remove them from the reported player count.

Honestly, it would be nice if there was an indicator for how many admins were online, and it could be separate from the player count still. Something where when you opened the dropdown for a server, it would have the server description, and then the number of admins online. That was this godsend can be implemented, and I can finally have a chance to hope on Harmony when there are 4 admins online (like I'm waiting for as I write this).\

Current: Screenshot from 2024-11-23 13-29-29 Concept: Admins online (GIMP FTW)

Wouldn't this also be a signal to raiders that they should go on servers with few admins online?

@ArtisticRoomba
Copy link
Contributor

most servers desire concealing the presence of admins
if raiders have an easy way to check if admins are online, it promotes people raiding at times when admins are offline
this can also extend to regular players deciding to skirt the rules or violate them because they saw that there are zero admins connected

making admins not count towards the playercap would actually be beneficial to admins as players could no longer half-guess when admins are on a server (a server at 81/80 players means that there has to be at least one admin online).

@ilovehans10
Copy link

most servers desire concealing the presence of admins if raiders have an easy way to check if admins are online, it promotes people raiding at times when admins are offline this can also extend to regular players deciding to skirt the rules or violate them because they saw that there are zero admins connected

making admins not count towards the playercap would actually be beneficial to admins as players could no longer half-guess when admins are on a server (a server at 81/80 players means that there has to be at least one admin online).

That totally makes sense, but why not just also make it a server option then. If you ask me with this option enabled it shouldn't show the admins in the player count, otherwise people won't know if there are spaces open once the server hits the cap. For example if a server is at 50/50 players online, but 5 of those players are admin and as such don't count towards the player count, most people would assume that the server is full even though it isn't.

The most server owner friendly option would probably be to provide a cvar for whether admins should be counted as players for the player count in the launcher.

@JIPDawg
Copy link
Contributor

JIPDawg commented Dec 10, 2024

Resolves #33424 I think

@Errant-4
Copy link
Member

Resolves #33424 I think

What issue where you trying to link? That's this PR

@TheShuEd TheShuEd merged commit d8afcdc into space-wizards:master Dec 16, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: Core Tech Area: Underlying core tech for the game and the Github repository. P1: High Priority: Higher priority than other items, but isn't an emergency. S: Approved Status: Reviewed and approved by at least one maintainer; a PR may require another approval. S: Needs Review Status: Requires additional reviews before being fully accepted size/S Denotes a PR that changes 10-99 lines. T: New Feature Type: New feature or content, or extending existing content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants