proof88 / PRooFPS-dd

PRooFPS-dd
https://www.facebook.com/whiskhyll
GNU General Public License v3.0
4 stars 1 forks source link

FTR: add CVAR for nIncomingMsgArraySize #227

Open proof88 opened 1 year ago

proof88 commented 1 year ago

It is inside PgeGsnWrapper::pollIncomingMessages(). Might be good for overcoming some issues sometimes. It controls the number of processed PgePackets per frame. Maybe we should calculate a theoretical max number of received PgePackets per frame in server and client cases and choose a number. We should also query the number of available message here from GNS and log it periodically. Not sure.

proof88 commented 1 year ago

Implement AP-7 - 12 as per PGE Networking wiki page to finish this task.

proof88 commented 12 months ago

Since PgeGnsWrapper::pollIncomingMessages() is invoked by a while loop until this returns true, there is no use for configuring this value, because it always fetches as many messages as possible that way until we run out of messages.

Instead we should monitor the max delay we (especially the server) spends in polling all messages, and maybe limit the max number of messages being polled per frame so server doesn't spend too much time in polling, since it also needs to send out packets at the end of frame to clients!