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 risk_tolerance + avoid overflow #5919

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ces42
Copy link
Contributor

@ces42 ces42 commented Mar 1, 2025

passed simplification STC:
LLR: 2.93 (-2.94,2.94) <-1.75,0.25>
Total: 73984 W: 19058 L: 18879 D: 36047
Ptnml(0-2): 232, 8735, 18866, 8940, 219
https://tests.stockfishchess.org/tests/view/67c269a38200cf1034c9baf9

passed simplification LTC:
LLR: 2.94 (-2.94,2.94) <-1.75,0.25>
Total: 39288 W: 10033 L: 9833 D: 19422
Ptnml(0-2): 14, 4168, 11086, 4356, 20
https://tests.stockfishchess.org/tests/view/67c34f8c8200cf1034c9bda1

Copy link

github-actions bot commented Mar 1, 2025

clang-format 18 needs to be run on this PR.
If you do not have clang-format installed, the maintainer will run it when merging.
For the exact version please see https://packages.ubuntu.com/noble/clang-format-18.

(execution 13625390164 / attempt 1)

bench: 2050046
@ces42
Copy link
Contributor Author

ces42 commented Mar 2, 2025

I'm confused, "Check includes" first told me to add #include <limits> and now it's telling me to remove it??

@ehsanrashid
Copy link

I'm confused, "Check includes" first told me to add #include <limits> and now it's telling me to remove it??

because you added it 2 times :)

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

Successfully merging this pull request may close these issues.

4 participants