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: Move Iteration respecting History Bonus #11

Merged
merged 4 commits into from
Jun 6, 2023

Conversation

TheBlackPlague
Copy link
Owner

🎯 Summary

This PR improves the history bonus applied for a beta cutoff move by having it depend on how later on the move was found to be the best. This is because if the move was found much later on, the history score of that move should be increased much more, and the score of all the moves that came before it should also be decreased much more depending on how early they were.

There are some caveats to this that need to be investigated since the moves are fairly early on in the list; should they turn out not to be the best, they would face serious score loss. But the hope is that it all balances out should they become relevant later on.

The formula change:

- HistoryBonus = Depth * Depth;
+ HistoryBonus = Depth * Depth + MoveIteration * MAX(Depth / 3, 1);

👏 Acknowledgements

NA

📈 ELO

STC:

ELO   | 6.56 +- 4.85 (95%)
SPRT  | 10.0+0.10s Threads=1 Hash=16MB
LLR   | 2.95 (-2.94, 2.94) [0.00, 5.00]
GAMES | N: 10640 W: 2981 L: 2780 D: 4879

LTC:

ELO   | 3.36 +- 2.68 (95%)
SPRT  | 60.0+0.60s Threads=1 Hash=256MB
LLR   | 2.96 (-2.94, 2.94) [0.00, 5.00]
GAMES | N: 32992 W: 8602 L: 8283 D: 16107

@TheBlackPlague TheBlackPlague added + ELO This change gains ELO. = DOC This change doesn't improve the documentation. labels Jun 6, 2023
@TheBlackPlague TheBlackPlague self-assigned this Jun 6, 2023
@TheBlackPlague TheBlackPlague marked this pull request as ready for review June 6, 2023 04:40
@TheBlackPlague TheBlackPlague merged commit 4786274 into master Jun 6, 2023
@TheBlackPlague TheBlackPlague deleted the history-move-iteration-dependent branch June 6, 2023 04:41
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