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

Add timestamp and/or PID to warning messages #772

Open
eduherminio opened this issue Aug 14, 2023 · 0 comments
Open

Add timestamp and/or PID to warning messages #772

eduherminio opened this issue Aug 14, 2023 · 0 comments

Comments

@eduherminio
Copy link

Output of cutechess-cli --version / cutechess --version

cutechess-cli 1.3.1
Using Qt version 5.15.2

How did you run Cute Chess exactly?

&"cutechess-1.3.1-win64\cutechess-cli.exe" `
-engine conf="Lynx-main" `
-engine conf="Lynx 0.15.0" `
-each `
    tc=inf/8+0.08 `
	option.Hash=64 `
-games 2 -rounds 20000 -repeat 2 -maxmoves 400 `
-openings file="books\UHO_XXL_+0.90_+1.19.epd" format=epd `
-sprt elo0=0 elo1=5 alpha=0.05 beta=0.05 `
-concurrency 8 `
-ratinginterval 10 `
-resultformat wide2 `
-pgnout "matches\tasks.pgn"

Expected behavior

Warning messages include a timestamp and with the process PID.

  • In order to debug warnings and errors, timestamp is critical to find them in our engines log files
  • In case of concurrency > 1, knowing the pid of the process where the issue happened would also help

Something like:

2023-08-14 11:16:23.7202: Warning: Illegal PV move d6c7 from Lynx-main (15754)
Warning: PV:  d6c7

Actual behavior

No timestamp and (n), which I'm not sure what it refers to (since n > concurrency)

Warning: Illegal PV move d6c7 from Lynx-main (12)
Warning: PV:  d6c7
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

No branches or pull requests

1 participant