Skip to content

Comments

Reformat elo change calculation methods#739

Open
kev306 wants to merge 17 commits intomasterfrom
feature/season-stat-main
Open

Reformat elo change calculation methods#739
kev306 wants to merge 17 commits intomasterfrom
feature/season-stat-main

Conversation

@kev306
Copy link
Collaborator

@kev306 kev306 commented Feb 10, 2025

No description provided.

@kev306 kev306 changed the title WIP: Reformat elo change calculation methdos Reformat elo change calculation methods Feb 24, 2025
Comment on lines +128 to +135
// TODO-kev: Is this check even required?
if (isNaN(user.roleStats[gameSizeKey][lowercaseRole].wins)) {
user.roleStats[gameSizeKey][lowercaseRole].wins = 0;
}

if (isNaN(user.roleStats[gameSizeKey][lowercaseRole].losses)) {
user.roleStats[gameSizeKey][lowercaseRole].losses = 0;
}
Copy link
Owner

Choose a reason for hiding this comment

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

yea you're probably right; probably can remove this

this.sendText(
'Error in elo calculation, no winning team specified.',
'server-text',
);
Copy link
Owner

@vck3000 vck3000 Feb 25, 2025

Choose a reason for hiding this comment

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

throw

}

finishGame(toBeWinner: Alliance) {
async finishGame(toBeWinner: Alliance) {
Copy link
Owner

Choose a reason for hiding this comment

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

Remove async

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants