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

TMM: Switch to new rating type format #1791

Closed
Askaholic opened this issue Jun 20, 2020 · 2 comments · Fixed by #2080
Closed

TMM: Switch to new rating type format #1791

Askaholic opened this issue Jun 20, 2020 · 2 comments · Fixed by #2080

Comments

@Askaholic
Copy link
Collaborator

Askaholic commented Jun 20, 2020

We are planning on moving rating data from the current tables to new more generic tables created in this PR: FAForever/db#197. The purpose of these new tables is to support more than 2 rating types, primarily for team matchmaking.

The client needs to be able to store arbitrarily many different rating types for each player. This means among other things, moving the globalRating and leaderboardRating properties on the Player class to some get/set table with string keys. Each rating type (referred to in the database as a leaderboard) has a technical name used to identify it. The technical names for global and ladder are global and ladder_1v1 respectively as defined here FAForever/db#217. We may want to add a KnownRatingType enum with these values as there are special cases for them all over the code.

Todo list:

Wanna have the bug fixed quickly?
Visit Issue hunt...
Issue hunt

@Askaholic Askaholic added this to To do in Team Matchmaker Jun 20, 2020
@MaxNF
Copy link

MaxNF commented Jun 20, 2020

I'm against spawning more KnownSmthType. You just wrote in Zulip:

For a similar reason I don’t think we want an enum for LadderQueue. Which queues are available depends entirely on the server. Some queues may not be permanent (special event queues that run for a month or whatever). At most I think it would be ok to have a special case for 1v1 because that is pretty much guaranteed to be permanent

That's why we do not need to hardcode rating type technical names either. What if we would like to display a rating for some temporary event queue? It will require changing the client code, that is not acceptable, right?
Maybe we can receive all ratings entirely from the server (or rest API) along with their parent queue titles and populate the UI with them?

@Askaholic
Copy link
Collaborator Author

Yes but currently global and ladder ratings have special cases everywhere. We won't be able to get rid of them in one pass, we will need to do it incrementally.

@Askaholic Askaholic moved this from To do to In progress in Team Matchmaker Dec 29, 2020
Team Matchmaker automation moved this from In progress to Done Jan 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging a pull request may close this issue.

2 participants