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

Extend by two if the side to move has only one legal move #4894

Closed
wants to merge 2 commits into from

Conversation

TarasVuk
Copy link
Contributor

This patch returns mate score instead of alpha in singular search.

Passed STC:
LLR: 2.97 (-2.94,2.94) <-1.75,0.25>
Total: 56576 W: 14374 L: 14172 D: 28030
Ptnml(0-2): 171, 6646, 14472, 6808, 191
https://tests.stockfishchess.org/tests/view/6564e3ab136acbc57354d443

Passed LTC:
LLR: 3.13 (-2.94,2.94) <-1.75,0.25>
Total: 103782 W: 25524 L: 25370 D: 52888
Ptnml(0-2): 58, 11753, 28118, 11901, 61
https://tests.stockfishchess.org/tests/view/6565e4c8136acbc57354ed23

bench: 1451915

bench: 1451915
@Disservin Disservin added the bench-change Changes the bench label Nov 29, 2023
bench: 1451915
@Disservin
Copy link
Member

I think this is a very hacky way and since it's a functional change and logic change, non regression bounds are not enough. It should also then be done differently instead of letting bestValue become a mate value.

@Disservin Disservin closed this Dec 9, 2023
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.

2 participants