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

Combo of various parameter tweaks. #3710

Closed

Conversation

Vizvezdenec
Copy link
Contributor

Passed STC
https://tests.stockfishchess.org/tests/view/6147deab3733d0e0dd9f313d
LLR: 2.94 (-2.94,2.94) <-0.50,2.50>
Total: 45488 W: 11691 L: 11450 D: 22347
Ptnml(0-2): 145, 5208, 11824, 5395, 172
Passed LTC
https://tests.stockfishchess.org/tests/view/6147f1d53733d0e0dd9f3141
LLR: 2.94 (-2.94,2.94) <0.50,3.50>
Total: 62520 W: 15808 L: 15482 D: 31230
Ptnml(0-2): 43, 6439, 17960, 6785, 33
Combination of parameter tweaks in search, evaluation and time management.
Original patches by @snicolet @xoto10 @lonfom169 and me.
Includes:

  1. Use bigger grain of positional evaluation more frequently (up to 1 exchange difference in non-pawn-material);
  2. More extra time according to increment;
  3. Increase margin for singular extensions;
  4. Do more aggresive parent node futility pruning.
    bench 5575265

@FauziAkram
Copy link
Contributor

FauziAkram commented Sep 20, 2021

Congratulations!
Well done!
By the way since in the combo there is also a change in the time management, do you think it needs some extra tests in some other time controls?

@Vizvezdenec
Copy link
Contributor Author

I'll leave it to maintainers to decide. Personally I don't really think so mainly because previous change of this sort passed all non-regression tests quite easily.

@xoto10
Copy link
Contributor

xoto10 commented Sep 20, 2021

I wondered this, but I think the question is what would we test? Sudden death shouldn't be affected because there's no increment, and tcec-style tc will have much less change than fishtest since the increment is 1/10 of the size. So maybe no other tests required.

@Vizvezdenec
Copy link
Contributor Author

Don't forget the more you test the bigger probability of false-negative result :)

@xoto10
Copy link
Contributor

xoto10 commented Sep 20, 2021

And there's that as well, you've convinced me! :-)

@vondele vondele closed this in e8788d1 Sep 21, 2021
Joachim26 pushed a commit to Joachim26/StockfishNPS that referenced this pull request Mar 28, 2023
Combination of parameter tweaks in search, evaluation and time management.
Original patches by snicolet xoto10 lonfom169 and Vizvezdenec.

Includes:

* Use bigger grain of positional evaluation more frequently (up to 1 exchange difference in non-pawn-material);
* More extra time according to increment;
* Increase margin for singular extensions;
* Do more aggresive parent node futility pruning.

Passed STC
https://tests.stockfishchess.org/tests/view/6147deab3733d0e0dd9f313d
LLR: 2.94 (-2.94,2.94) <-0.50,2.50>
Total: 45488 W: 11691 L: 11450 D: 22347
Ptnml(0-2): 145, 5208, 11824, 5395, 172

Passed LTC
https://tests.stockfishchess.org/tests/view/6147f1d53733d0e0dd9f3141
LLR: 2.94 (-2.94,2.94) <0.50,3.50>
Total: 62520 W: 15808 L: 15482 D: 31230
Ptnml(0-2): 43, 6439, 17960, 6785, 33

closes official-stockfish#3710

bench 5575265
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.

None yet

3 participants