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

Skip early pruning in singular search #4881

Closed

Conversation

TarasVuk
Copy link
Contributor

This patch skips early pruning and updating of history based on static eval difference in singular search.

Passed STC:
LLR: 2.93 (-2.94,2.94) <-1.75,0.25>
Total: 387872 W: 97721 L: 97890 D: 192261
Ptnml(0-2): 1202, 46691, 98263, 46634, 1146
https://tests.stockfishchess.org/tests/view/6552660c136acbc57353490a

Passed LTC:
LLR: 2.95 (-2.94,2.94) <-1.75,0.25>
Total: 127518 W: 31399 L: 31285 D: 64834
Ptnml(0-2): 80, 14442, 34606, 14546, 85
https://tests.stockfishchess.org/tests/view/655502e5136acbc573537b9b

bench: 1498102

bench: 1498102
@Disservin
Copy link
Member

I'm not particular fond of introducing a new goto jump. It also skips Probcut entirely, whereas before it only skipped a matching move inside probcut.

@Disservin Disservin added the bench-change Changes the bench label Nov 20, 2023
@vondele
Copy link
Member

vondele commented Nov 20, 2023

I also don't really see this as a simplification patch.

@Disservin
Copy link
Member

Yeah, I'm closing it, thanks. Interestingly one might try to skip probcut in case there's a excluded move, with gainer bounds perhaps?

@Disservin Disservin closed this Nov 20, 2023
@peregrineshahin
Copy link
Contributor

Yeah, I'm closing it, thanks. Interestingly one might try to skip probcut in case there's a excluded move, with gainer bounds perhaps?

Was tried few days ago by cj5.

@TarasVuk
Copy link
Contributor Author

I also don't really see this as a simplification patch.

I also wasn't sure about it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bench-change Changes the bench
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants