Skip to content
This repository has been archived by the owner on Oct 21, 2023. It is now read-only.

mod: only enable rating in battle #1145

Merged
merged 3 commits into from
Jul 13, 2023
Merged

mod: only enable rating in battle #1145

merged 3 commits into from
Jul 13, 2023

Conversation

verdie-g
Copy link
Owner

Conquest can be a cluster fuck and rating gains/loses are not very relevant so I suggest to disable it. Rating will still be used for team balancing though.

Conquest can be a cluster fuck and rating gains/loses are not very
relevant so I suggest to disable it. Rating will still be used for
team balancing though.
@verdie-g verdie-g added enhancement New feature or request area/mod Mod related labels Jul 13, 2023
@verdie-g verdie-g requested a review from namidaka July 13, 2023 03:16
Copy link
Collaborator

@namidaka namidaka left a comment

Choose a reason for hiding this comment

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

Shouldn't Scoreboard also be fed enableRating to disable Rating Calculation in ScoreHit as this could be deemed to be a hot path?

@verdie-g
Copy link
Owner Author

The OnScoreHit is actually done in CrpgRewardServer, you can see I'm using the boolean in that method.

@verdie-g verdie-g merged commit a85ae08 into master Jul 13, 2023
@verdie-g verdie-g deleted the rating-battle-only branch July 13, 2023 17:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/mod Mod related enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants