Does not understand simple seki

Issue #11 resolved

dsmic created an issue 2012-02-17

it plays last2libatari by putting itself into selfatari, therefore it thinks the c7 group is dead:(

Comments (7)

dsmic reporter
This is a situation, where isSelfAtariOfSize is still buggy:(
- 2012-02-17T16:38:37+00:00
dsmic reporter
- attached Go.cc
I think there is wrong historical code in the function, can you have a look at? I commented it out. I think it does not handle correctly, if two touching strings have the same liberty, this is handled already by the code before the outcommented part?!

(I was not sure enough what I did to commit, therefore I just attach the source file)
- 2012-02-17T17:14:00+00:00
Francois van Niekerk repo owner
- changed status to open
I will definitely have a look. Thank you for looking into it.
- 2012-02-17T17:15:35+00:00
Francois van Niekerk repo owner
- changed status to resolved
This seems to be fixed after the latest big merge.
- 2012-07-04T12:10:49+00:00
dsmic reporter
Might be a good idea to collect this kind of situations as a quality check.

We might construct a test suite to take care that we do not introduce the problems again?!

We might have a look at the test cases posted on computer go last month too...
- 2012-07-04T14:53:04+00:00
Francois van Niekerk repo owner
These should be added to the regression tests in the "regression" folder. I plan to add regression tests for a number of bugs after I have fixed them.
- 2012-07-04T15:14:55+00:00
dsmic reporter
Oh, great. I did not use it up to now. At the moment there are some regressiong failing:)
- 2012-07-04T16:48:34+00:00
Log in to comment