Lobby - Slow rating updates break rejoining

[elexis]

Since some days there are experimental lobby changes to reduce the updates that the bot sends. This definitely helps with the performance, but it has a side effect.

It takes like 2-3 seconds for the ratinglist to appear.

• If you join a game before the ratinglist is loaded, you will join under "username".
• If you join a game after the ratinglist is loaded, you will join as "username (rating)".

This means that it's sometimes impossible to rejoin games without understanding this bug.

---

Solution: Send the ratinglist / rating for the current user instantaneously and/or prohibit lobby joins before the rating is loaded.

[Josh]

It was non-optimal to include the rating in the username in the first place. It should be an independent variable. For example, I don't believe the current system allows players to rejoin lobby games by direct IP.

[leper]

Replying to Josh:

I don't believe the current system allows players to rejoin lobby games by direct IP.

It does though.

[elexis]

Backlogging due to lack of progress.

[elexis]

Still happens with #3022, just less frequently.

[scythetwirler]

Should happen even less often (if at all) with the latest optimizations to the bot.

[elexis]

Still occurs as of Alpha 21. It can be solved by the client clicking on his own user profile before joining, so it should be fixable by issuing that request on init in lobby.js, or finding some bug in XPartaMupp.py (which would require ejabberd to debug).

[Josh]

I am still of the opinion that the rating should be stored separately. Adding a rating lookup function to the C++ side would allow for that.

[elexis]

This is happening too often.

[shookees]

Games shouldn't be identified by name, especially when it is not constant

[elexis]

refs #4621

[elexis]

In 19828:

Allow lobby players to rejoin games, even if the rating changed meanwhile or if the lobby bot didn't provide the most recent rating.

Refs #3686
Differential Revision: ​https://code.wildfiregames.com/D670
Reviewed By: Imarok

[elexis]

With r19828 the three cases where players can't rejoin games anymore are covered:

• lobby bot being too slow with providing players the ratinglist on init or not providing it at all
• #4621: lobby bot not providing the updated ratinglist to players returning from a rated game
• Also if a player plays a rated game and rejoins a game started before then works.

So it is not imminent to fix anymore. But the original issue of the lobby bot being too slow described here is probably still not fixed. We should report whether we could still experience the issue.

Removing the rating from the nickname has the advantage of being able to remove that 4 line workaround, but will also need many new lines in all GUI pages that display playernames (gamesetup, session, summary, replaymenu) and sounds like material for a separate ticket. However replacing the simulation playername with a GUI generated one will be needed for #3307.

[elexis]

Calling this fixed.

Removing the rating from the nick must be carefully considered and should be done in a separate ticket.

The bot is now allowed to be slow, the other reported bug is reported at #4621.