-
-
Notifications
You must be signed in to change notification settings - Fork 25
mod : ClanGroup Size Penalty moved to server configuration #1204
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Team balancing is what killed the AS server so we need to find a safest plan
Moved it to server configuration. So we can change it for a single day on a server by server basis , and run analysis of rounds for each server to tweak the penalty on a per server basis. |
|| !float.TryParse(sizePenaltyStr, out float sizePenalty) | ||
|| sizePenalty > 1.5f) | ||
{ | ||
Debug.Print($"Invalid server multiplier: {sizePenaltyStr}"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To update
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Up
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
@@ -25,11 +25,28 @@ public static void Init() | |||
public static CrpgRegion Region { get; } | |||
public static string Service { get; } | |||
public static string Instance { get; } | |||
public static float ClanGroupSizePenalty { get; private set; } = 0f; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since it's a global variable it's important to give as much context as possible in the name. We often say
The length of the identifier should be proportional to the distance between its declaration and use.
So here you could go with TeamBalancerClanGroupSizePenalty
and crpg_team_balancer_clan_group_size_penalty
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
e2fd767
to
4783cec
Compare
No description provided.