farnulfo / ai-contest

Automatically exported from code.google.com/p/ai-contest
0 stars 0 forks source link

Patch for visualizer so it merges fleets #148

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
For example, see here: http://ai-contest.com/visualizer.php?game_id=4462846: 
guylimajr breaks all the fleets he sends into smaller fleets with size of one 
(which isn't exactly a good thing, but anyway), and in the visualizer they are 
all displayed separately. Thus, user only sees the topmost one, and can only 
wonder what's going on. Or, alternatively, see the raw game_info output, which 
isn't pleasant, either.

I wrote a patch that enables the visualizer to merge fleets that were sent at 
the same turn and have same sources and destinations

Original issue reported on code.google.com by gvsmirno...@gmail.com on 14 Sep 2010 at 11:52

GoogleCodeExporter commented 8 years ago
Oh, the patch

Original comment by gvsmirno...@gmail.com on 14 Sep 2010 at 11:52

Attachments:

GoogleCodeExporter commented 8 years ago
Maybe this merging should be done in the engine itself ? Because if not, that 
would be a good way to slow down the enemy with a flood of fleets. (ultimate 
attack : launch N fleets :D)

Original comment by Maxim...@gmail.com on 14 Sep 2010 at 7:16

GoogleCodeExporter commented 8 years ago
Maxime81, you are right, and this has already been mentioned in issue 112. 
There are, however, many already saved replays that have the problem. So it's 
either running a script on all replays or just merging them in the visualizer.

Original comment by gvsmirno...@gmail.com on 14 Sep 2010 at 7:37

GoogleCodeExporter commented 8 years ago
Yes, as you said, the engine now merges fleets. I think that's the right place 
for it.

As new submissions are made, old games are made less relevant, and won't even 
be linked to from the site. So I'm not going to worry too much about 
visualizing them.

Original comment by danie...@gmail.com on 20 Sep 2010 at 12:35