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.

Sign up for GitHub

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

Jump to bottom

cutechess doesn't send null string in engine options #663

Open
voyag opened this issue Apr 25, 2021 · 2 comments · May be fixed by #712
Open

cutechess doesn't send null string in engine options #663

voyag opened this issue Apr 25, 2021 · 2 comments · May be fixed by #712

Comments

@voyag
Copy link

voyag commented Apr 25, 2021 •
edited

`

stockfish_210226(0): setoption name Analysis Contempt value Both
stockfish_210226(0): setoption name Contempt value 24
stockfish_210226(0): setoption name Debug Log File
stockfish_210226(0): setoption name EvalFile value nn-62ef826d1a6d.nnue
stockfish_210226(0): setoption name Hash value 16
stockfish_210226(0): setoption name Move Overhead value 10
stockfish_210226(0): setoption name MultiPV value 1
stockfish_210226(0): setoption name Skill Level value 20
stockfish_210226(0): setoption name Slow Mover value 100
stockfish_210226(0): setoption name Syzygy50MoveRule value true
stockfish_210226(0): setoption name SyzygyPath
stockfish_210226(0): setoption name SyzygyProbeDepth value 1
stockfish_210226(0): setoption name SyzygyProbeLimit value 7
stockfish_210226(0): setoption name Threads value 1
stockfish_210226(0): setoption name UCI_Elo value 1350
stockfish_210226(0): setoption name UCI_LimitStrength value false
stockfish_210226(0): setoption name Use NNUE value true
stockfish_210226(0): setoption name nodestime value 0
`

This is the initialization log of an engine the string options with string "" doesn't have "value".
For example
setoption name Debug Log File
should be
setoption name Debug Log File value
the engines usually expect the "value" to detect the new initialization even if it is empty
alwey added a commit to alwey/cutechess that referenced this issue Dec 30, 2021
@alwey
For UCI command setoption send keyword value even if null value follows
799fbed 
Resolves cutechess#663
@alwey
Copy link
Contributor

alwey commented Dec 30, 2021

I am not sure what is worse - not sending value or sending value with no argument in tow. I prepared a patch in my repository to send value for all kinds of option types but not for button options - see above.

alwey added a commit to alwey/cutechess that referenced this issue Feb 5, 2022
@alwey
For UCI command setoption send keyword value even if null value follows
6018b5d 
Resolves cutechess#663
@alwey
Copy link
Contributor

alwey commented Feb 5, 2022

@voyag Thanks for reporting, I am now uploading a pull request with a fix.

@alwey alwey linked a pull request Feb 5, 2022 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

For UCI command "setoption" send keyword "value" even if null value follows alwey/cutechess
2 participants
@alwey @voyag