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 accumulator storage/updates #5927

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

xu-shawn
Copy link
Contributor

@xu-shawn xu-shawn commented Mar 13, 2025

Passed Non-regression STC:
LLR: 2.93 (-2.94,2.94) <-1.75,0.25>
Total: 115840 W: 29983 L: 29854 D: 56003
Ptnml(0-2): 338, 12990, 31149, 13091, 352
https://tests.stockfishchess.org/tests/view/67d0a044166a3e8781d84223

no functional change

@xu-shawn
Copy link
Contributor Author

#5912 might need rebasing on this PR

no functional change
no functional change
@Disservin Disservin requested a review from Sopel97 March 13, 2025 07:12
@mstembera
Copy link
Contributor

Why is this better? I mean it's almost 300 extra lines more.

@Sopel97
Copy link
Member

Sopel97 commented Mar 13, 2025

I like that there's finally some effort to decouple accumulator from position, I'll try to review this later when I have time.

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.

3 participants