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

Time management parameter changes #4843

Closed

Conversation

XInTheDark
Copy link
Contributor

@XInTheDark XInTheDark commented Oct 23, 2023

  1. Tune time management parameters.
  2. Scale the optimum time and maximum time parameters based on the amount of time left, using a logarithmic scale.

Many acknowledgements to @FauziAkram for tuning the parameters and for the original idea (see https://tests.stockfishchess.org/tests/view/652f0356de6d262d08d333c5).

STC: https://tests.stockfishchess.org/tests/view/6533938fde6d262d08d39e4d
LLR: 2.94 (-2.94,2.94) <0.00,2.00>
Total: 44320 W: 11301 L: 10982 D: 22037
Ptnml(0-2): 146, 4810, 11920, 5147, 137

LTC: https://tests.stockfishchess.org/tests/view/653477e4de6d262d08d3ae06
LLR: 2.95 (-2.94,2.94) <0.50,2.50>
Total: 146442 W: 37338 L: 36811 D: 72293
Ptnml(0-2): 60, 14975, 42645, 15460, 81

Verification runs:
3+0.03: https://tests.stockfishchess.org/tests/view/65364e7ef127f3553505178a
10+0: https://tests.stockfishchess.org/tests/view/65364e9ff127f3553505178f
180+1.8: https://tests.stockfishchess.org/tests/view/65364ec3f127f35535051794

Bench: 1241996

@vondele
Copy link
Member

vondele commented Oct 23, 2023

can you run some additional tests of this PR on different TCs, as we have done for time mgmt patches before (non-regression is fine). In particular, I'd like to see what happens at short TC (e.g. 3+0.03), sudden death (10+0), and VLTC (180+1.8).

@FauziAkram
Copy link
Contributor

I don't think it's cool to snipe someone's idea while they're still testing it. I've taken inspiration from other developers too, but I always make sure they've finished all their testing first, and I usually wait a few months after their last test just to be sure they're not onto something related.

Right now, I've got an LTC in progress, and I'm waiting to see if it works out or not before trying some other stuff (I was about to try a version very similar to what you did). So, it's kind of a bummer when someone uses my idea while I'm still in the middle of testing and tweaking.

I hope next time you can able more patient and let the developer finish all their tests before we jump in on their ideas. It's just fair play.

@Disservin Disservin added the to be merged Will be merged shortly label Oct 23, 2023
@Disservin Disservin closed this in cf3dbcb Oct 23, 2023
@Disservin
Copy link
Member

@FauziAkram I've added you as a Co-Author to this PR, see cf3dbcb. Given your influence on it.
Maybe first communicate with @FauziAkram on discord @XInTheDark and coordinate your ideas ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 gainer Gains elo no-bench-change to be merged Will be merged shortly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants