Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Not sure about this IsEyeShape() value is intended or a bug #36

Closed
geovens opened this issue Nov 3, 2017 · 2 comments
Closed

Not sure about this IsEyeShape() value is intended or a bug #36

geovens opened this issue Nov 3, 2017 · 2 comments

Comments

@geovens
Copy link

geovens commented Nov 3, 2017

2017-11-03 21-25-09

In this corner, IsSelfAtariNadake(S19) is true as expected. What I wonder about is the fact that IsEyeShape(S19) is true. Is this intended? If there was a black stone at T18, IsEyeShape(19) is false as it should be.
In fact, in this corner white should play S19 to kill black, but AQ excludes S19 as a candidate child node because it is considered an eyeshape.

@ymgaq
Copy link
Owner

ymgaq commented Nov 8, 2017

Your point is right.

This is not a bug but specification. AQ is designed to ignore some shapes with low appearance probability for speeding up. If I try to recognize this shape as SelfAtariNakade, the code becomes complicated and slows down.

Please tell me if you have a good idea to modify this.

@ymgaq
Copy link
Owner

ymgaq commented Jan 14, 2018

This issue is closed due to lack of recent activity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants