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.

Sign up for GitHub

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

Jump to bottom

Simplify internal iterative reductions #5634

Closed

TarasVuk wants to merge 1 commit into official-stockfish:master from TarasVuk:iir_cutnode_ttmove_pr

Contributor

TarasVuk commented Oct 14, 2024

Passed STC:
LLR: 2.92 (-2.94,2.94) <-1.75,0.25>
Total: 138656 W: 36182 L: 36074 D: 66400
Ptnml(0-2): 523, 16422, 35310, 16570, 503
https://tests.stockfishchess.org/tests/view/6702beb386d5ee47d953bd41

Passed LTC:
LLR: 2.94 (-2.94,2.94) <-1.75,0.25>
Total: 680844 W: 172021 L: 172480 D: 336343
Ptnml(0-2): 492, 76259, 187419, 75720, 532
https://tests.stockfishchess.org/tests/view/67042b1f86d5ee47d953be7c

bench: 1294546


          Update search.cpp

c694630

bench: 1294546

Disservin added the simplification label

Contributor

xu-shawn commented Oct 24, 2024 •

edited

Loading

Since depth-- patches like this tend to scale badly, maybe we should verify that this PR doesn't regress at VLTC?

Disservin closed this in

bf2a0d5

Disservin pushed a commit that referenced this pull request


          Revert VLTC regression from #5634

54cf226

https://tests.stockfishchess.org/tests/view/671bf61b86d5ee47d953cf23

And thanks to @xu-shawn for suggesting running a VLTC regress test since
depth modifications affect scaling. Also, the LTC was showing a slight
regress after 680+k games  ~= -0.34 , for reference:
https://tests.stockfishchess.org/tests/view/67042b1f86d5ee47d953be7c

closes #5663

Bench: 1307308

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels