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

Stop lmr reduction from overflowing #241

Merged
merged 1 commit into from
Jul 3, 2024
Merged

Conversation

sroelants
Copy link
Owner

Not expecting this to affect much, but will at least do a regression test, I suppose?

Looks like maybe 5 elo?

Elo   | 5.09 +- 5.60 (95%)
SPRT  | 8.0+0.08s Threads=1 Hash=16MB
LLR   | 2.94 (-2.94, 2.94) [-5.00, 0.00]
Games | N: 7298 W: 2245 L: 2138 D: 2915
Penta | [234, 802, 1482, 885, 246]
https://chess.samroelants.com/test/39/

bench 4956902

Not expecting this to affect much, but will at least do a regression
test, I suppose?

bench 4956902
@sroelants sroelants merged commit b7d752b into main Jul 3, 2024
2 checks passed
@sroelants sroelants deleted the search/fix-lmr-overflows branch July 3, 2024 16:58
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.

None yet

1 participant