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

Set the SF16 pure NNUE reference nps wrt SF11 #1746

Merged
merged 1 commit into from
Jul 12, 2023

Conversation

ppigazzini
Copy link
Collaborator

Fishtest with Stockfish 11 had 1.6Mnps as reference nps and 0.7Mnps as threshold for the slow worker.
Set the new reference nps to 0.64Mnps according to the 60% slowdown of the SF 16 wrt SF 11 on the ARCH=x86-64-bmi2
Set the new threshold for slow worker to 0.231Mnps according to the 67% slowdown of the SF 16 wrt SF 11 on the ARCH=x86-64-sse41-popcnt

compiler clang++ 16.0.6

  • arch=bmi2 (Intel Xeon CPU E5-2680 v3)
sf_base =  1515099 +/-   6276 (95%)
sf_test =   610755 +/-   2686 (95%)
diff    =  -904343 +/-   5940 (95%)
speedup = -59.689% +/- 0.392% (95%)
  • arch=sse41-popcnt (Intel core i7 3770k)
sf_base =  1929595 +/-  19733 (95%)
sf_test =   654161 +/-   5421 (95%)
diff    = -1275433 +/-  14999 (95%)
speedup = -66.099% +/- 0.777% (95%)

Raise worker version to 207, also server side.

Fishtest with Stockfish 11 had 1.6Mnps as reference nps
and 0.7Mnps as threshold for the slow worker.
Set the new reference nps to 0.64Mnps according to the 60% slowdown
of the SF 16 wrt SF 11 on the ARCH=x86-64-bmi2
Set the new threshold for slow worker to 0.231Mnps according to
the 67% slowdown of the SF 16 wrt SF 11 on the ARCH=x86-64-sse41-popcnt

compiler clang++ 16.0.6

- arch=bmi2 (Intel Xeon CPU E5-2680 v3)
```
sf_base =  1515099 +/-   6276 (95%)
sf_test =   610755 +/-   2686 (95%)
diff    =  -904343 +/-   5940 (95%)
speedup = -59.689% +/- 0.392% (95%)
```

- arch=sse41-popcnt (Intel core i7 3770k)
```
sf_base =  1929595 +/-  19733 (95%)
sf_test =   654161 +/-   5421 (95%)
diff    = -1275433 +/-  14999 (95%)
speedup = -66.099% +/- 0.777% (95%)
```

Raise worker version to 207, also server side.
@ppigazzini ppigazzini added worker update code changes requiring a worker update update code change to account external change (Stockfish, book etc.) labels Jul 12, 2023
@ppigazzini ppigazzini merged commit a72b999 into official-stockfish:master Jul 12, 2023
17 checks passed
@ppigazzini ppigazzini deleted the pure_nuue_nps branch July 12, 2023 11:34
@ppigazzini
Copy link
Collaborator Author

PROD updated, triggered the worker update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
update code change to account external change (Stockfish, book etc.) worker update code changes requiring a worker update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant