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

replace depth increase condition with !opponentWorsening #5798

Closed
wants to merge 1 commit into from

Conversation

Ergodice
Copy link
Contributor

Replace depth increase condition with !opponentWorsening

Passed simplification STC
LLR: 2.94 (-2.94,2.94) <-1.75,0.25>
Total: 220544 W: 57417 L: 57399 D: 105728
Ptnml(0-2): 816, 26554, 55540, 26520, 842
https://tests.stockfishchess.org/tests/view/678970e38082388fa0cbfe02

Passed simplification LTC
LLR: 2.94 (-2.94,2.94) <-1.75,0.25>
Total: 132600 W: 33868 L: 33760 D: 64972
Ptnml(0-2): 126, 14770, 36390, 14898, 116
https://tests.stockfishchess.org/tests/view/678accabc00c743bc9e9fc7f

bench 1517399

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

@vondele
Copy link
Member

vondele commented Jan 18, 2025

@Ergodice is this the right PR, diff seems a bit large?

@xu-shawn
Copy link
Contributor

xu-shawn commented Jan 18, 2025

i think it's a PR on top of #5797 #5785

@vondele vondele added simplification A simplification patch to be merged Will be merged shortly labels Jan 18, 2025
@vondele vondele closed this in 8e3e22b Jan 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
simplification A simplification patch to be merged Will be merged shortly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants