Simplify Time Management #5076
Simplify Time Management #5076
Conversation
bench: 1303971
|
I think it's much better to let the VLTC test finish. Non-regression can't be statistically proved without the actual LLR. |
|
@XInTheDark Ok, resumed it, and now completed (Passed). |
Disservin
added
no-bench-change
simplification
|
can you check which commit introduced this, and see what the testcase was? It was almost certainly introduced for a case where the increment is large compared to the base time, and none of these tests test this. |
|
If I see this correctly, it was introduced here: #3702 Actually, there was no test with a larger increment than in LTC. |
|
But probably the special case was : "TCEC Style 10+0.01 " |
@vondele https://tests.stockfishchess.org/tests/view/65e4ca73416ecd92c162a57d |
|
Passed also the TCEC TC style suggested by vondele: |
Disservin
added
to be merged
Simplify Time Management
Instead of having a formula for using extra time with larger increments.
Simply set it to 1 when the increment is lower than 0.5s and to 1.1 when the increment is higher.
The values can later on be further improved.
Passed STC:
LLR: 2.93 (-2.94,2.94) <-1.75,0.25>
Total: 27488 W: 7077 L: 6848 D: 13563
Ptnml(0-2): 96, 3041, 7267, 3218, 122
https://tests.stockfishchess.org/tests/view/65d25d3c1d8e83c78bfd9293
Passed LTC:
LLR: 2.94 (-2.94,2.94) <-1.75,0.25>
Total: 137568 W: 34612 L: 34512 D: 68444
Ptnml(0-2): 60, 14672, 39221, 14770, 61
https://tests.stockfishchess.org/tests/view/65d2a72c1d8e83c78bfd97fa
Passed VLTC:
LLR: 2.94 (-2.94,2.94) <-1.75,0.25>
Total: 139650 W: 35229 L: 35134 D: 69287
Ptnml(0-2): 33, 14227, 41218, 14306, 41
https://tests.stockfishchess.org/tests/view/65d7d7d39b2da0226a5a205b
Passed also the TCEC TC style suggested by vondele:
LLR: 2.94 (-2.94,2.94) <-1.75,0.25>
Total: 134150 W: 34278 L: 34163 D: 65709
Ptnml(0-2): 561, 15727, 34444, 15722, 621
https://tests.stockfishchess.org/tests/view/65e4ca73416ecd92c162a57d