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

Refactor root history into low ply history. #5614

Closed

Conversation

Vizvezdenec
Copy link
Contributor

This patch changes root history to low ply history - butterfly history for plies < 4.
Doubles weight of this history for root, latter plies have lesser effect.
Passed STC:
https://tests.stockfishchess.org/tests/view/66f77d2386d5ee47d953b65d
LLR: 2.94 (-2.94,2.94) <0.00,2.00>
Total: 180992 W: 47362 L: 46830 D: 86800
Ptnml(0-2): 554, 21499, 45928, 21891, 624
Passed LTC:
https://tests.stockfishchess.org/tests/view/66fb557986d5ee47d953b8e5
LLR: 2.95 (-2.94,2.94) <0.50,2.50>
Total: 42462 W: 11013 L: 10682 D: 20767
Ptnml(0-2): 33, 4518, 11795, 4855, 30
bench 1498677

Copy link

github-actions bot commented Oct 2, 2024

clang-format 18 needs to be run on this PR.
If you do not have clang-format installed, the maintainer will run it when merging.
For the exact version please see https://packages.ubuntu.com/noble/clang-format-18.

(execution 11160440456 / attempt 1)

@Disservin Disservin added the 🚀 gainer Gains elo label Oct 2, 2024
@pb00067
Copy link

pb00067 commented Oct 3, 2024

The color-dimension is not needed in the lowPlyHistory, it can be simplified away (the ply dimension is sufficient)

P.S.: Now I saw its has already been noticed by another guy on discord. sorry for the noise

@Vizvezdenec
Copy link
Contributor Author

I think you meant guy :)

removing colour
removing colour
removing colour
adding a comment
@vondele vondele added the to be merged Will be merged shortly label Oct 4, 2024
@vondele vondele closed this in 7ac745a Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 gainer Gains elo to be merged Will be merged shortly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants