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 #5216

Conversation

XInTheDark
Copy link
Contributor

@XInTheDark XInTheDark commented May 5, 2024

This patch is the result of two tuning stages:

  1. ~32k games at 60+0.6 th8: https://tests.stockfishchess.org/tests/view/662d9dea6115ff6764c7f817
  2. ~193k games at 80+0.8 th6, based on PR Introduce Quadruple Extensions #5211: https://tests.stockfishchess.org/tests/view/663587e273559a8aa857ca00. Based on extensive VVLTC tuning and testing both before and after Introduce Quadruple Extensions #5211, it is observed that introduction of new extensions positively affected the search tune results.

Passed VVLTC 70+0.7 th7 1st sprt: https://tests.stockfishchess.org/tests/view/6636c6f04b68b70d85801409
LLR: 2.94 (-2.94,2.94) <0.00,2.00>
Total: 18566 W: 4864 L: 4620 D: 9082
Ptnml(0-2): 0, 1608, 5827, 1844, 4

Passed VVLTC 70+0.7 th7 2nd sprt: https://tests.stockfishchess.org/tests/view/6636d4b84b68b70d85802ab7
LLR: 2.94 (-2.94,2.94) <0.50,2.50>
Total: 43142 W: 11141 L: 10838 D: 21163
Ptnml(0-2): 4, 3915, 13427, 4224, 1

Passed VVLTC 70+0.7 3rd sprt: https://tests.stockfishchess.org/tests/view/66376b4f9819650825aa230b
LLR: 2.94 (-2.94,2.94) <0.50,2.50>
Total: 40322 W: 10374 L: 10076 D: 19872
Ptnml(0-2): 1, 3660, 12544, 3952, 4

The first two sprts were run against passed #5211. The third sprt was run against latest master.

Bench: 2180675

@vondele vondele added the 🚀 gainer Gains elo label May 5, 2024
@vondele
Copy link
Member

vondele commented May 5, 2024

please rebase on master, and test once more. You can use throughput 400.

Bench: 2180675
@XInTheDark
Copy link
Contributor Author

VVLTC has finished, I've updated the PR description and rebased

@Disservin Disservin added the to be merged Will be merged shortly label May 5, 2024
@Disservin
Copy link
Member

btw this isnt entirely clean rebased.. ideally it should be squashed and/or all commits should be above the latest one which is in master. The history looks like a merge

image

@Disservin Disservin closed this in 070e564 May 5, 2024
@Disservin
Copy link
Member

Rebased it correctly locally and merged.
I am running this when merging a branch into master locally, --ff-only basically ensures that the commit history is linear, which is what we want. git merge --ff-only vvltc_search_tune_040524_4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 gainer Gains elo to be merged Will be merged shortly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants