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 improvement term in null move search #4650

Closed

Conversation

XInTheDark
Copy link
Contributor

@XInTheDark XInTheDark commented Jul 1, 2023

Simply remove improvement term in null move search.

Simplification STC: https://tests.stockfishchess.org/tests/view/649c0d2edc7002ce609d33b5
LLR: 2.95 (-2.94,2.94) <-1.75,0.25>
Total: 271104 W: 72181 L: 72217 D: 126706
Ptnml(0-2): 691, 30042, 74129, 29992, 698

Simplification LTC: https://tests.stockfishchess.org/tests/view/649d0dd7dc7002ce609d4efa
LLR: 2.94 (-2.94,2.94) <-1.75,0.25>
Total: 183120 W: 49469 L: 49418 D: 84233
Ptnml(0-2): 84, 17636, 56063, 17699, 78

Bench: 2613939

Bench: 2736274
@cj5716
Copy link
Contributor

cj5716 commented Jul 1, 2023

I think you wrote the wrong bench
image

@XInTheDark
Copy link
Contributor Author

It seems to be some error with GitHub actions, I can confirm on local compilation and fishtest that the provided bench is correct

@vondele vondele added the to be merged Will be merged shortly label Jul 3, 2023
@vondele vondele closed this in eb9aaf9 Jul 3, 2023
Joachim26 pushed a commit to Joachim26/StockfishNPS that referenced this pull request Jul 31, 2023
passed STC:
https://tests.stockfishchess.org/tests/view/649c0d2edc7002ce609d33b5
LLR: 2.95 (-2.94,2.94) <-1.75,0.25>
Total: 271104 W: 72181 L: 72217 D: 126706
Ptnml(0-2): 691, 30042, 74129, 29992, 698

passed LTC:
https://tests.stockfishchess.org/tests/view/649d0dd7dc7002ce609d4efa
LLR: 2.94 (-2.94,2.94) <-1.75,0.25>
Total: 183120 W: 49469 L: 49418 D: 84233
Ptnml(0-2): 84, 17636, 56063, 17699, 78

closes official-stockfish#4650

Bench: 2642851
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
to be merged Will be merged shortly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants