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

bench: 5046381 #3666

Closed

Conversation

candirufish
Copy link
Contributor

LMR Pv and depth 6 Extension tweak:

LTC:
LLR: 2.93 (-2.94,2.94) <0.50,3.50>
Total: 52488 W: 1542 L: 1394 D: 49552
Ptnml(0-2): 18, 1253, 23552, 1405, 16

https://tests.stockfishchess.org/tests/view/611e49c34977aa1525c9caa7

STC:
LLR: 2.94 (-2.94,2.94) <-0.50,2.50>
Total: 76216 W: 6000 L: 5784 D: 64432
Ptnml(0-2): 204, 4745, 28006, 4937, 216

https://tests.stockfishchess.org/tests/view/611e0e254977aa1525c9ca89

bench: 5046381

@Vizvezdenec
Copy link
Contributor

I would recommend you to change title to smth more of a general description.
Like "do more LMR extensions for PV nodes"
And provide better description in a PR like what this patch actually does :)

@anshulongithub
Copy link

Does the comment above the changed line need to be updated to state the new logic? Just wanted to confirm.

@Vizvezdenec
Copy link
Contributor

Yeah, it should be slightly updated.

@vondele vondele added the to be merged Will be merged shortly label Aug 22, 2021
@vondele vondele closed this in 939ffe4 Aug 22, 2021
@candirufish
Copy link
Contributor Author

Was away for a while, well I see it got merge already oh well...

Joachim26 pushed a commit to Joachim26/StockfishNPS that referenced this pull request Mar 28, 2023
LMR Pv and depth 6 Extension tweak:

LTC:
LLR: 2.93 (-2.94,2.94) <0.50,3.50>
Total: 52488 W: 1542 L: 1394 D: 49552
Ptnml(0-2): 18, 1253, 23552, 1405, 16
https://tests.stockfishchess.org/tests/view/611e49c34977aa1525c9caa7

STC:
LLR: 2.94 (-2.94,2.94) <-0.50,2.50>
Total: 76216 W: 6000 L: 5784 D: 64432
Ptnml(0-2): 204, 4745, 28006, 4937, 216
https://tests.stockfishchess.org/tests/view/611e0e254977aa1525c9ca89

closes official-stockfish#3666

Bench: 5046381
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
to be merged Will be merged shortly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants