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 lmr condition #5582

Conversation

Nonlinear2
Copy link
Contributor

Remove the moveCount increase in the LMR condition.

Passed non-regression STC:
LLR: 2.93 (-2.94,2.94) <-1.75,0.25>
Total: 87104 W: 22630 L: 22464 D: 42010
Ptnml(0-2): 316, 10295, 22132, 10525, 284
https://tests.stockfishchess.org/tests/view/66dccd00dc53972b68218c60

Passed non-regression LTC:
LLR: 2.94 (-2.94,2.94) <-1.75,0.25>
Total: 94050 W: 23869 L: 23722 D: 46459
Ptnml(0-2): 49, 10400, 25985, 10537, 54
https://tests.stockfishchess.org/tests/view/66dd69c7dc53972b68218ca5

Bench: 1394597

@Disservin
Copy link
Member

when you have please add the bench, btw are you in the authors file?

Bench: 1394597
@Nonlinear2
Copy link
Contributor Author

I've amended the commit to include the bench in the commit message, so the checks should work now. And yes, i'm already in the authors file.

@Disservin Disservin added simplification A simplification patch functional-change to be merged Will be merged shortly labels Sep 10, 2024
@Disservin Disservin closed this in d8e49cd Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
functional-change 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