Skip to content

feat: add log level trait and structs and fix info output - #150

Merged
ptsouchlos merged 5 commits into
mainfrom
feature/improve-bench-output
Oct 17, 2025
Merged

feat: add log level trait and structs and fix info output#150
ptsouchlos merged 5 commits into
mainfrom
feature/improve-bench-output

Conversation

@ptsouchlos

Copy link
Copy Markdown
Owner

Changes

  • Control log level and output from the engine when searching. Same approach is used in toad.
  • Don't output any search info during the search bench - instead pretty print the results
  • Fixed a bug with how we output info lines. In certain cases (very low time control), no info line was being emitted which was causing the issue in the UCI check in feat(ci): add uci compliance check #113 .

This will let us control log level and output from the engine when
searching. Same approach is used in
[toad](https://github.com/dannyhammer/toad).
This also fixes an issue with output a proper info line when the search
time is very short. This was causing a bug with the UCI check as part of
#22 and #113
Use logging for the search bench and by default don't output anything.
Instead, pretty format the progress of the search bench.

bench: 1041224
@ptsouchlos

Copy link
Copy Markdown
Owner Author
Elo   | -0.46 +- 1.64 (95%)
SPRT  | 8.0+0.08s Threads=1 Hash=16MB
LLR   | 2.99 (-2.94, 2.94) [-5.00, 0.00]
Games | N: 45124 W: 7197 L: 7257 D: 30670
Penta | [398, 4404, 12960, 4460, 340]
https://pyronomy.pythonanywhere.com/test/3257/

@ptsouchlos
ptsouchlos merged commit 827bbcc into main Oct 17, 2025
17 checks passed
@ptsouchlos
ptsouchlos deleted the feature/improve-bench-output branch October 17, 2025 17:21
ptsouchlos pushed a commit that referenced this pull request Nov 5, 2025
Elo   | -0.46 +- 1.64 (95%)
SPRT  | 8.0+0.08s Threads=1 Hash=16MB
LLR   | 2.99 (-2.94, 2.94) [-5.00, 0.00]
Games | N: 45124 W: 7197 L: 7257 D: 30670
Penta | [398, 4404, 12960, 4460, 340]
https://pyronomy.pythonanywhere.com/test/3257/

bench: 1041224
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.

1 participant