Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Time management refactoring & bugfix #136

Merged
merged 1 commit into from
Jun 11, 2021
Merged

Conversation

QueensGambit
Copy link
Owner

Refactored time management and fixed bug when losing on time.
Now it is taken into account that increment is applied after the move is done (and not before).

Thanks to AdminX and Werner Schüle for reporting it:


./cutechess-cli -variant standard -openings file="/data/SL/opening_suites/UHO_V3_6mvs_+090_+099.pgn" format=pgn plies=100 -pgnout "/data/SL/pgn_out/time_manag_refactor.pgn" -resign movecount=5 score=600 -draw movenumber=30 movecount=4 score=20 -concurrency 1 -engine name=ClassicAra-0.9.5_time_refactor_70e37e5 cmd=/data/SL/@JC_ClassicAra_time_refactor_70e37e5 -engine name=ClassicAra-0.9.5_e17e608_wdlp_input_3.0 cmd=/data/SL/@JC_ClassicAra_input_3.0 -each dir=/data/SL/ option.Threads=3 proto=uci tc=0/0:30+0.25 option.Model_Directory=/data/SL/model/input_3.0/7_wdlp_head_7_epochs option.First_Device_ID=15 option.Last_Device_ID=15 -games 2 -rounds 300 -repeat&

TC: 30s +  0.25s
File: /data/SL/pgn_out/time_manag_refactor.pgn

Name                        :  Games                 Elo    Pts   Pts%   Win  Loss  Draw  Tf  Sc
ClassicAra-0.9.5_e17e608_wd :    600     1.16 +/-  18.08  301.0   50.2   128   126   346   0   0
ClassicAra-0.9.5_time_refac :    600    -1.16 +/-  18.08  299.0   49.8   126   128   346   0   0

Name                        :  W_win  B_win   Draw
ClassicAra-0.9.5_e17e608_wd :    123      5    346
ClassicAra-0.9.5_time_refac :    117      9    346


Name                        :   Wapc     Sd
ClassicAra-0.9.5_e17e608_wd :    130     35
ClassicAra-0.9.5_time_refac :    131     38

Finished games   : 600
White wins       : 240 (40.0%)
Black wins       : 14 (2.3%)
Draws            : 692 (115.3%)
Unfinished games : 0 (0.0%)

Tf = time forfeit
Sc = stalled connection
Wapc = win average ply count, lower is better
Sd = standard deviation

take into account that increment is applied after the move is done (and not before)
@QueensGambit QueensGambit changed the title Time management refactoring Time management refactoring & bugfix Jun 11, 2021
@QueensGambit QueensGambit merged commit 51ef9f2 into master Jun 11, 2021
QueensGambit added a commit that referenced this pull request Jun 16, 2021
@QueensGambit QueensGambit deleted the time_manag_refactor branch June 30, 2021 11:44
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant