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

VVLTC search tune #5803

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

Conversation

XInTheDark
Copy link
Contributor

@XInTheDark XInTheDark commented Jan 19, 2025

Parameters were tuned with 75k VVLTC games: https://tests.stockfishchess.org/tests/view/678b0e5ac00c743bc9ea0135.

Passed VVLTC sprt 1: https://tests.stockfishchess.org/tests/view/678b9326c00c743bc9ea0f2f
LLR: 2.94 (-2.94,2.94) <0.50,2.50>
Total: 33866 W: 8913 L: 8630 D: 16323
Ptnml(0-2): 2, 2994, 10659, 3275, 3

Passed VVLTC sprt 2: https://tests.stockfishchess.org/tests/view/678bca3ef4dc0a8b4ae8cd95
LLR: 2.95 (-2.94,2.94) <0.00,2.00>
Total: 42258 W: 10969 L: 10695 D: 20594
Ptnml(0-2): 2, 3801, 13253, 4067, 6

Passed VVLTC sprt 3 (rebased): https://tests.stockfishchess.org/tests/view/678c60f8f4dc0a8b4ae8db9f
LLR: 2.95 (-2.94,2.94) <0.50,2.50>
Total: 60822 W: 15851 L: 15514 D: 29457
Ptnml(0-2): 4, 5572, 18925, 5903, 7

Bench: 1633960

Copy link

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 12850958264 / attempt 1)

@XInTheDark XInTheDark mentioned this pull request Jan 20, 2025
@vondele vondele added the to be merged Will be merged shortly label Jan 25, 2025
@@ -34,7 +34,7 @@ using Range = std::pair<int, int>; // Option's min-max values
using RangeFun = Range(int);

// Default Range function, to calculate Option's min-max values
inline Range default_range(int v) { return v > 0 ? Range(0, 2 * v) : Range(2 * v, 0); }
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

contains changes that should probably not be merged.
Please rebase after the next merge wave, and squash in a single commit.

@vondele vondele removed the to be merged Will be merged shortly label Jan 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants