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

🦈 IMP: Add a King SEE Value #32

Merged
merged 2 commits into from
Jul 26, 2023
Merged

🦈 IMP: Add a King SEE Value #32

merged 2 commits into from
Jul 26, 2023

Conversation

TheBlackPlague
Copy link
Owner

🎯 Summary

The current SEE values used to approximate SEE are missing a value for the King Piece. While typically not required, this value can be a good addition for future accurate SEE algorithms. This PR adds that value.

👏 Acknowledgements

NA

📈 ELO

STC:

ELO   | 1.21 +- 2.79 (95%)
SPRT  | 10.0+0.10s Threads=1 Hash=16MB
LLR   | 2.96 (-2.94, 2.94) [-3.00, 1.00]
GAMES | N: 30504 W: 7910 L: 7804 D: 14790

LTC:

ELO   | 0.36 +- 2.02 (95%)
SPRT  | 60.0+0.60s Threads=1 Hash=256MB
LLR   | 2.96 (-2.94, 2.94) [-3.00, 1.00]
GAMES | N: 53968 W: 12899 L: 12843 D: 28226

@TheBlackPlague TheBlackPlague added + ELO This change gains ELO. = DOC This change doesn't improve the documentation. labels Jul 26, 2023
@TheBlackPlague TheBlackPlague self-assigned this Jul 26, 2023
@TheBlackPlague TheBlackPlague marked this pull request as ready for review July 26, 2023 12:58
@TheBlackPlague TheBlackPlague merged commit afc1bea into master Jul 26, 2023
@TheBlackPlague TheBlackPlague deleted the see-approx-king branch July 26, 2023 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
= DOC This change doesn't improve the documentation. + ELO This change gains ELO.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant