Show correct player count for lobbies

Show correct player count for lobbies

Postby ThomasHiatt » 27 Jan 2018, 10:06

I suggest that the client should actually show the number of people in a lobby instead of saying there are more than there really are. It is pretty annoying to have your lobby appear full or over-full just because someone was kicked or disconnected and it didn't affect the player count.
ThomasHiatt
Avatar-of-War
 
Posts: 78
Joined: 02 Feb 2017, 00:24
Has liked: 33 times
Been liked: 24 times
FAF User Name: ThomasHiatt

Re: Show correct player count for lobbies

Postby Wesmania » 27 Jan 2018, 17:05

Unfortunately the client doesn't have a say in this, it just shows what the server tells us. That's a bug for server people to hunt down and squash :(
(No longer) FAF python client dev. You client misbehaves? I might help if in a good mood.
Wesmania
Priest
 
Posts: 336
Joined: 19 Nov 2014, 19:17
Has liked: 1 time
Been liked: 57 times
FAF User Name: MazorNoob

Re: Show correct player count for lobbies

Postby Brutus5000 » 11 Feb 2018, 17:52

That's a general design issue in the client-server-protocol. The server does not send single game updates, but all game infos in a list. If the server would send the game list more often, it will cause server performance issues eventually.

Changing to an event-based communication is a huge refactoring both client and server side. We could tackle this with the a completely new protocol. Maybe based on websockets as is under development in the java server. But with all respect due, the python client is still not showing any visible progress regarding the backend improvements of the last 12 month (i.e. new map, mod and replay vault). So I don't think this is happening any time soon.
Want to know what I'm working on? Browse my public task queue

The 9th Doctor wrote:You think it'll last forever, the people and cars and concrete. But it won't. One day it's all gone, even the sky.
Brutus5000
Avatar-of-War
 
Posts: 176
Joined: 05 Aug 2016, 23:32
Has liked: 19 times
Been liked: 170 times
FAF User Name: brutus5000

Re: Show correct player count for lobbies

Postby Wesmania » 11 Feb 2018, 18:48

The issue in question is not about player counts being outdated, but about the server sometimes sending game info with the ingame player number higher than the max player number. Is this something that's affected by batching game updates? I don't see why it would.
(No longer) FAF python client dev. You client misbehaves? I might help if in a good mood.
Wesmania
Priest
 
Posts: 336
Joined: 19 Nov 2014, 19:17
Has liked: 1 time
Been liked: 57 times
FAF User Name: MazorNoob

Re: Show correct player count for lobbies

Postby Brutus5000 » 11 Feb 2018, 23:05

Not sure what is meant here then.

Are we talking about a case where the game list shows 11 / 10 players? Or are we talking about a case where 6/8 players is shown, but the tooltip lists a different amount of players? Or what exactly is the problem? A screenshot would be helpful.
Want to know what I'm working on? Browse my public task queue

The 9th Doctor wrote:You think it'll last forever, the people and cars and concrete. But it won't. One day it's all gone, even the sky.
Brutus5000
Avatar-of-War
 
Posts: 176
Joined: 05 Aug 2016, 23:32
Has liked: 19 times
Been liked: 170 times
FAF User Name: brutus5000

Re: Show correct player count for lobbies

Postby ThomasHiatt » 12 Feb 2018, 05:29

Say there is a lobby that was just hosted with 1/6 players. If someone joins this lobby it will immediately show 2/6 on the play tab in the client. Then 4 more people join and it says 6/6. If the host kicks 2 people who are AFK it will still show 6/6 on the play tab in the client, I believe it says that until the players who were kicked click the ok button in the dialog box saying they were kicked. Now the lobby looks like it is full, or over-filled, when in reality you are still needing players to join it.
ThomasHiatt
Avatar-of-War
 
Posts: 78
Joined: 02 Feb 2017, 00:24
Has liked: 33 times
Been liked: 24 times
FAF User Name: ThomasHiatt

Re: Show correct player count for lobbies

Postby Brutus5000 » 12 Feb 2018, 21:44

ThomasHiatt wrote:If the host kicks 2 people who are AFK it will still show 6/6 on the play tab in the client, I believe it says that until the players who were kicked click the ok button in the dialog box saying they were kicked.


That makes sense. Since everything in the lobby is peer-to-peer the server doesn't know about the kick until the kicked player's game closes the connection to the server. This probably happens once the player clicks the ok button. No chance for the server to know it earlier than that, without changing the game protocol and opening the gates for all sorts of mischievous activities.
Want to know what I'm working on? Browse my public task queue

The 9th Doctor wrote:You think it'll last forever, the people and cars and concrete. But it won't. One day it's all gone, even the sky.
Brutus5000
Avatar-of-War
 
Posts: 176
Joined: 05 Aug 2016, 23:32
Has liked: 19 times
Been liked: 170 times
FAF User Name: brutus5000


Return to FAF Suggestions

Who is online

Users browsing this forum: No registered users and 1 guest