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

Remove -stat_malus(newDepth) #5631

Closed
wants to merge 1 commit into from

Conversation

FauziAkram
Copy link
Contributor

@FauziAkram FauziAkram commented Oct 12, 2024

Simplification on top of PR #5621
Remove -stat_malus(newDepth)

Passed STC (Tested against PR 5621):
LLR: 2.93 (-2.94,2.94) <-1.75,0.25>
Total: 32544 W: 8448 L: 8221 D: 15875
Ptnml(0-2): 112, 3813, 8192, 4046, 109
https://tests.stockfishchess.org/tests/view/6709178286d5ee47d953c148

Passed LTC (Tested against PR 5621):
LLR: 2.94 (-2.94,2.94) <-1.75,0.25>
Total: 41046 W: 10391 L: 10189 D: 20466
Ptnml(0-2): 25, 4497, 11284, 4685, 32
https://tests.stockfishchess.org/tests/view/6709579886d5ee47d953c16a

bench: 1137315

bench: 1137315
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 11304748032 / attempt 1)

@mstembera
Copy link
Contributor

I don't think the tests are against the correct version of PR #5621.

@FauziAkram
Copy link
Contributor Author

FauziAkram commented Oct 12, 2024

@mstembera That's right, Fixing it

@FauziAkram FauziAkram closed this Oct 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants