[Freeciv-Dev] Re: [Patch][RFC] Reduce superfluous tile_info packets

[Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx>]

On Wed, Oct 24, 2001 at 05:51:12PM -0700, Raahul Kumar wrote:
> 
> --- Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx> wrote:
> > 
> > The attached patch is a rough first version of a patch which change
> > the server to only sent tile information if this information has
> > really changed.
> > 
> > Test: loading a normal game (1600AD) and pressing three times return.
> > Results (packet type 7 is tile info):
> > 
> >   without patch:
> > 2:   [ 7]:  41882 packets;   376938 bytes total;     9 bytes/packet average
> > 2: transmitted 534864 bytes in 44000 packets;average size per packet 12 
> > bytes
> > 
> >   with patch:
> > 2:   [ 7]:   4076 packets;    36684 bytes total;     9 bytes/packet average

Amount of bytes goes down to 9.7%

> > 2: transmitted 185739 bytes in 6000 packets;average size per packet 30 bytes

Amount of bytes goes down to 34.7%

> Impressive. What about throught out the game? I suspect your patch should
> reduce the amount of packets overall. It would be nice to see a graph
> of total amount of packets sent before and packets after.

I won't collect the numbers for various stages of the game. This is
burdensome and quite useless IMHO. However I can give another data
point (same as above but with more turns):

  without patch:
2:   [ 7]:  87708 packets;   789372 bytes total;     9 bytes/packet average
2: received 1091048 bytes in 92000 packets;average size per packet 11 bytes

  with patch:
2:   [ 7]:   4251 packets;    38259 bytes total;     9 bytes/packet average

Amount of bytes goes down to 4.8%

2: received 304028 bytes in 8000 packets;average size per packet 38 bytes

Amount of bytes goes down to 27.8%

So this will yield better result with longer games.

With the patch the packet type which uses the most bytes is city_info:

2:   [11]:     42 packets;    48474 bytes total;  1154 bytes/packet average

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
 "Life is too short for reboots."