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

Extended search #305

Closed
wants to merge 5 commits into from
Closed

Extended search #305

wants to merge 5 commits into from

Conversation

ujh
Copy link
Owner

@ujh ujh commented Aug 4, 2016

If the best move selected by win ratio and number of plays don't agree continue searching.

Benchmarks are still outstanding to see if this even improves the strength or not.

Closes #209

ujh added 3 commits August 3, 2016 17:00
This also changes the code to use the node with the most plays instead
of the best win ratio
@ujh
Copy link
Owner Author

ujh commented Aug 4, 2016

Whoops. It seems that some tests are still failing!

ujh added 2 commits August 4, 2016 14:42
Instead just calculate the result twice. This is not in the part of the
program that needs to be fast anyway. And doing it like this makes the
code more understandable.
@ujh
Copy link
Owner Author

ujh commented Aug 8, 2016

It turns out that this doesn't help at all. Even with a value of 10 it doesn't play better than the version without it.

@ujh ujh closed this Aug 8, 2016
@ujh ujh deleted the extended-search branch August 8, 2016 05:28
@ujh ujh removed the 3 - Review label Aug 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant