Re: [gnugo-devel] Inge's patch (fix)

[Gunnar Farneback]

Inge wrote:
> I didn't know this.  That explains a lot.  I worked with standard
> command line switches and that works as it would in a game using ASCII
> mode, for instance.  Does this mean that Gnu Go might play differently
> on NNGS (using the GTP client) than in a personal game (using cgoban
> and GMP)?  If so, I don't think that is a good thing.  If it is only
> for regressions, then fine, but during games, the behaviour should be
> the same. 

Of course gnugoclient uses the genmove_* gtp commands which do
generate empty threats, since this is intended for actual games.

> > And indeed your bugfix does solve the problem also with
> > filllib:15. Run the filllib test suite and you will see that it all
> > passes again.
> 
> Beleive me, I have run the tests many times.

Not sufficiently it seems. Well, I didn't actually apply your bugfix
patch, only fix the obvious bug in genmove_conservative(), then the
test suite passed. But I don't really think there's anything else in
your last patch that could cancel that bugfix.

> But in the general case it is not that easy.  I found it useful, and I
> think that I could have used it a number of times in the past too.

Absolutely. But maybe it would be better to reuse the --color option
also for this purpose?

/Gunnar