Re: [gnugo-devel] paul_3_15.7

[Paul Pogonyshev]

Dan wrote:
> I'm planning on adding paul_3_15.5 later. Originally
> my testing seemed to show that this patch is slow.
>
> But I now think that is not true. Inge has pointed out
> that my method of timing is flawed. I just timed it a
> third time and found no appreciable slowdown. (The
> first timing can be ignored because I was running
> gnome at the time.) My latest experiment has it
> running in 9080 seconds, only 65 seconds slower than
> the last time I ran the regressions. (Unfortunately
> this also includes Gunnar's patch, but it's unlikely
> that Gunnar's patch is a speedup.)
>
> Paul pointed out to me that there isn't really anything
> in the patch that should make it slow.

the only reason i can see is a possible increase in owl nodes.
i'm rather busy currently, but i think i can run regressions
and test this. just tell me where Gunnar's script which can
count them is.

> About the patch paul_3_15.7, this adds a line to
> patterns/Makefile:
>
> josekidb.c : $(DBBUILT) mkpat$(EXEEXT)
>          cat  $(DBBUILT) | ./mkpat -b joseki >josekidb.c
>          cat  $(DBBUILT) | ./mkpat -C joseki_c >>josekidb.c

that part of the patch is not for cvs as i wrote in the mail. it
is intended for testing only and i included it so you can see how
the matcher works. until the matcher is connected to a callback
function, i don't think we should generate new database and put
it into executable.

> After that we get a bunch of error messages:
>
> <stdin>(3042) : warning : duplicated patterns encountered (JH177 and JH180)
> <stdin>(4684) : warning : duplicated patterns encountered (JH261 and JH280)
> <stdin>(4797) : warning : duplicated patterns encountered (JH213 and JH287)
> <stdin>(4813) : warning : duplicated patterns encountered (JH214 and JH288)
> <stdin>(4831) : warning : duplicated patterns encountered (JH215 and JH289)
> <stdin>(4847) : warning : duplicated patterns encountered (JH216 and JH290)
> <stdin>(4865) : warning : duplicated patterns encountered (JH217 and JH291)
> <stdin>(5833) : warning : duplicated patterns encountered (JH336 and JH348)
> <stdin>(5851) : warning : duplicated patterns encountered (JH337 and JH349)
> <stdin>(5867) : warning : duplicated patterns encountered (JH338 and JH350)
> <stdin>(5885) : warning : duplicated patterns encountered (JH339 and JH351)
> <stdin>(5901) : warning : duplicated patterns encountered (JH340 and JH352)
> <stdin>(5919) : warning : duplicated patterns encountered (JH341 and JH353)
> <stdin>(5935) : warning : duplicated patterns encountered (JH342 and JH354)
> <stdin>(5953) : warning : duplicated patterns encountered (JH343 and JH355)
> <stdin>(5969) : warning : duplicated patterns encountered (JH344 and JH356)
> <stdin>(5987) : warning : duplicated patterns encountered (JH345 and JH357)
> <stdin>(8685) : warning : duplicated patterns encountered (JH159 and JH526)
>
> I don't understand why the patterns are added twice. In any case
> let's leave this patch until after the 3.3.15 release.

there are some duplicated variations in hoshi.sgf. i'll check it
once i'm done with my exams.

Paul