Caching in accumulateMaterials()

mannyzhou5 / evolutionchamber

Automatically exported from code.google.com/p/evolutionchamber

0 stars 0 forks source link

Caching in accumulateMaterials() #149

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

I added some dumb caching around mineMinerals() and mineGas() that increases 
games played/sec by as much as 40%.  

It also looks like there was a bug in worker parity, taking Math.min(s.bases() 
* 16, 3), when I assume it was meant to be Math.min(s.bases(), 3) * 16.

Original issue reported on code.google.com by eton...@gmail.com on 14 Nov 2010 at 1:39

Attachments:

cachingAccumulateMaterials.patch

GoogleCodeExporter commented 9 years ago

Original comment by netpr...@gmail.com on 16 Nov 2010 at 10:12

Added labels: Performance
Removed labels: ****

GoogleCodeExporter commented 9 years ago

Applied the patch to trunk. Thanks! :)

Original comment by netpr...@gmail.com on 16 Nov 2010 at 6:21

Changed state: Fixed
Added labels: Type-Enhancement
Removed labels: Type-Defect

GoogleCodeExporter commented 9 years ago

netprobe, see my commit review.

Original comment by mike.angstadt on 16 Nov 2010 at 9:18

Added labels: ****
Removed labels: ****

GoogleCodeExporter commented 9 years ago

Replied
https://code.google.com/p/evolutionchamber/source/detail?r=112

Original comment by netpr...@gmail.com on 16 Nov 2010 at 9:48

Added labels: ****
Removed labels: ****