[Freeciv-Dev] [bug #18767] cma: always 1 tax collector

2013-02-19 Thread Bastian Schmidt
Follow-up Comment #20, bug #18767 (project freeciv): Just to clarify, the CMA does not look at the value of gold_upkeep_style. I have not looked at, why gold_upkeep_style=2 could resolve the issue (as described in comment #13), as the flawed pruning caused (at least in my game) many unnecessary

[Freeciv-Dev] [patch #3620] Governor (cma) branch pruning heuristic fix

2013-02-05 Thread Bastian Schmidt
Follow-up Comment #5, patch #3620 (project freeciv): - I believe it is not necessary to initialize production[] (bug #20482). It is either copied from soln-production or solnplus.production (in compute_max_stats_heuristic) - You are right, city production should be restored. There are some

[Freeciv-Dev] [patch #3620] Governor (cma) branch pruning heuristic fix

2013-01-31 Thread Bastian Schmidt
Follow-up Comment #2, patch #3620 (project freeciv): Good point, I just had the civ2 ruleset in mind. I have revised the patch and added a new version. The hardcoded twos had to be replaced. Everything is now as in city.c. (file #17079)

[Freeciv-Dev] [patch #3620] Governor (cma) branch pruning heuristic fix

2013-01-25 Thread Bastian Schmidt
URL: http://gna.org/patch/?3620 Summary: Governor (cma) branch pruning heuristic fix Project: Freeciv Submitted by: stlth Submitted on: Fr 25 Jan 2013 14:16:10 GMT Category: agents Priority: 5 - Normal

[Freeciv-Dev] [bug #18767] cma: always 1 tax collector

2013-01-25 Thread Bastian Schmidt
Follow-up Comment #17, bug #18767 (project freeciv): I have taken a detailed look at the cma branch bound algorithm, and found the implemented pruning to be severely flawed. IMHO this is the cause for the described problems. I submitted patch #3620 , see there for additional details.