(PR#10203) Greedy CM algorithm

"Vasco Alexandre da Silva Costa" <[email protected]> Sat, 13 Nov 2004 18:55:51 -0800
Newsgroups gmane.games.freeciv.ai
Message-ID <[email protected]>
<URL: http://rt.freeciv.org/Ticket/Display.html?id=10203 >

> [jdorje - Sat Nov 13 06:18:14 2004]:
> 
> I can't say I entirely understand it.  But then the current code is
> even worse.  It does seem pretty clear that the B&B algorithm will
> be a lot less bug-prone than the current method.  So unless anyone
> objects I'll commit this patch (Benoit, make more changes to it if
> you want).
> 
> For S2_0 this is too much of a change, even though it's surely easier
> to debug than the DP algorithm.  So we still want bug fixes for the
> current system.

I beg to disagree. This code is plug-in compatible with the old and
doesn't change the network protocol, i18n strings or anything. It is
totally self-contained in one file (cm.c).

It fixes longstanding bugs and is faster to boot. I think we should
put it in S2_0 now. If it turns out it has even more bugs than the old
one, it is a simple matter of switching it back. Even if this does
happen, we will have got the benefit of increased testing coverage,
which will facilitate its development into a more robust form.