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: Accurate SEE (Static Exchange Evaluation) #33

Merged
merged 6 commits into from
Jul 29, 2023
Merged

Conversation

TheBlackPlague
Copy link
Owner

🎯 Summary

This PR removes the Approximation function used for Static Exchange Evaluation (SEE) pruning with an Accurate function. The goal is that this allows for a deeper understanding of good and bad captures, avoiding bad captures that may seem good initially.

The algorithm used is the SWAP Algorithm.

The implementation changes the SEE Pruning currently done in QSearch to use the Accurate function instead of the Approximate one.

👏 Acknowledgements

  • Jay Honnold: The algorithm implementation is largely inspired by Jay's implementation of the algorithm. Kudos, Jay. 🙌

📈 ELO

STC:

ELO   | 21.32 +- 9.75 (95%)
SPRT  | 10.0+0.10s Threads=1 Hash=16MB
LLR   | 2.98 (-2.94, 2.94) [0.00, 5.00]
GAMES | N: 2480 W: 706 L: 554 D: 1220

LTC:

ELO   | 18.56 +- 8.99 (95%)
SPRT  | 60.0+0.60s Threads=1 Hash=256MB
LLR   | 2.96 (-2.94, 2.94) [0.00, 5.00]
GAMES | N: 2848 W: 782 L: 630 D: 1436

@TheBlackPlague TheBlackPlague added + ELO This change gains ELO. = DOC This change doesn't improve the documentation. labels Jul 29, 2023
@TheBlackPlague TheBlackPlague self-assigned this Jul 29, 2023
@TheBlackPlague TheBlackPlague marked this pull request as ready for review July 29, 2023 14:22
@TheBlackPlague TheBlackPlague merged commit b73f52f into master Jul 29, 2023
@TheBlackPlague TheBlackPlague deleted the see branch July 29, 2023 14:27
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