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

Do not use SEE in evasion scoring #819

Merged
merged 1 commit into from
Oct 6, 2016

Conversation

snicolet
Copy link
Member

@snicolet snicolet commented Oct 5, 2016

Idea by Aram Tumanian (atumanian)

STC:
LLR: 2.95 (-2.94,2.94) [-3.00,1.00]
Total: 43889 W: 7849 L: 7767 D: 28273

LTC:
LLR: 2.95 (-2.94,2.94) [-3.00,1.00]
Total: 29333 W: 3809 L: 3700 D: 21824

Bench: 6421663

Idea by Aram Tumanian (atumanian)

STC:
LLR: 2.95 (-2.94,2.94) [-3.00,1.00]
Total: 43889 W: 7849 L: 7767 D: 28273

LTC:
LLR: 2.95 (-2.94,2.94) [-3.00,1.00]
Total: 29333 W: 3809 L: 3700 D: 21824

Bench: 6421663
@snicolet
Copy link
Member Author

snicolet commented Oct 5, 2016

@atumanian: I supposed that your complete name is "Aram Tumanian" based on your github login (atumanian) and the beginning of your gmail address as shown in the forum ([email protected]). Is that guess correct? Congrats for the original idea! :-)

@atumanian
Copy link

atumanian commented Oct 5, 2016

Yes, it's my name. Thank you!
The idea appeared while I was working on optimisation of the Position:see function. I had even decided to run your exact test after my test would have completed.

@atumanian
Copy link

I've created a pull request for my optimisation which includes your patch: #820

@mcostalba mcostalba merged commit 1e58628 into official-stockfish:master Oct 6, 2016
@mcostalba
Copy link

Merged with 1e58628

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

Successfully merging this pull request may close these issues.

3 participants