server icon indicating copy to clipboard operation
server copied to clipboard

Matchmaker rating shown as 0

Open BlackYps opened this issue 3 years ago • 8 comments

The client shows no 4v4 matchmaker rating for many people when they play their first game even though the initialization happens correctly on the server. So it defaults to 0 in game. Oddly enough for some people it works. Correct rating initialization can be confirmed by looking at the ratings of the replay after the game has finished

BlackYps avatar Dec 19 '21 23:12 BlackYps

This happens because the server does the initialization internally and doesn't send the client the players rating before the game starts so the client defaults to zero.

So this isn't really a client issue because it doesn't have the necessary information

Sheikah45 avatar Dec 19 '21 23:12 Sheikah45

Then why does it work for some players?

BlackYps avatar Dec 19 '21 23:12 BlackYps

Largely this is probably just a race condition which is why it works for some people

Sheikah45 avatar Dec 19 '21 23:12 Sheikah45

Also, @Askaholic says they are in the player_info message, so the client can get them from there. I suspect that is why it works for some people, depending on when the message arrives

BlackYps avatar Dec 19 '21 23:12 BlackYps

The player_info message is not sent out right as soon as a change occurs. The server batches up the players and sends then in intervals. So that is the race condition. If it gets an updated message before the game starts it will show the initialized rating. If it doesn't then the players will show as 0.

Sheikah45 avatar Dec 19 '21 23:12 Sheikah45

Makes sense. Probably the right thing to do would be to add the rating to the game_launch message?

Askaholic avatar Dec 19 '21 23:12 Askaholic

for who all the players just the player who the message is going to? Also I will move this to the server

Sheikah45 avatar Dec 19 '21 23:12 Sheikah45

This could be made irrelevant with https://github.com/FAForever/fa/pull/6160 because then the bug doesn't show up anymore

BlackYps avatar May 07 '24 23:05 BlackYps