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

Simplify away capthist bonus in Probcut #5761

Closed
wants to merge 2 commits into from

Conversation

xu-shawn
Copy link
Contributor

@xu-shawn xu-shawn commented Jan 11, 2025

The explicit bonus has been obsoleted with the introduction of #5695

Passed Non-regression STC:
LLR: 2.95 (-2.94,2.94) <-1.75,0.25>
Total: 132832 W: 34519 L: 34403 D: 63910
Ptnml(0-2): 430, 15754, 33931, 15872, 429
https://tests.stockfishchess.org/tests/view/678158c49168c8bf30927834

Passed Non-regression LTC:
LLR: 2.95 (-2.94,2.94) <-1.75,0.25>
Total: 123492 W: 31426 L: 31309 D: 60757
Ptnml(0-2): 79, 13705, 34051, 13842, 69
https://tests.stockfishchess.org/tests/view/6782b07e6ddf09c0b4b6dbb7

bench 1088944

bench 1088944
@Disservin
Copy link
Member

search.cpp:867:20: error: variable ‘captured’ set but not used [-Werror=unused-but-set-variable]
  867 |         Piece      captured;

@Disservin Disservin added simplification A simplification patch to be merged Will be merged shortly labels Jan 11, 2025
@Disservin Disservin closed this in 9213618 Jan 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
simplification A simplification patch to be merged Will be merged shortly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants