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

case sensitive UCI option names #683

Open
jdart1 opened this issue Oct 12, 2021 · 1 comment · May be fixed by #692
Open

case sensitive UCI option names #683

jdart1 opened this issue Oct 12, 2021 · 1 comment · May be fixed by #692

Comments

@jdart1
Copy link

jdart1 commented Oct 12, 2021 •
edited

I have cutechess-cli 1.1, platform is Linux x64 (Ubuntu). Not sure if this bug is still in current versions, but the issue is, options specified on the command line seem to be case-sensitive, while the UCI protocol spec specifies they should be case-insensitive.

For example:

-engine conf=$conf option.Threads=1 option.Hash=128 option."NNUE file"=arasan-d8-9-20210827.nnue

works, while

-engine conf=$conf option.Threads=1 option.Hash=128 option."NNUE File"=arasan-d8-9-20210827.nnue

fails with an error:

Warning: Arasan-23.0 doesn't have option NNUE File

The UCI option is declared on startup to be "NNUE file" (no capitalized f). I.e. Arasan outputs "option name NNUE file type string default".
@alwey
Copy link
Contributor

alwey commented Dec 25, 2021

Confirmed. Same with version 1.2.0 and current master.

alwey added a commit to alwey/cutechess that referenced this issue Dec 25, 2021
@alwey
Make UCI options case insensitive
e088e3c 
Resolves cutechess#683
@alwey alwey linked a pull request Dec 25, 2021 that will close this issue
Open
alwey added a commit to alwey/cutechess that referenced this issue Jan 22, 2022
@alwey
Make UCI options case insensitive
292fecd 
Resolves cutechess#683
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.

Make UCI options case insensitive alwey/cutechess
2 participants
@jdart1 @alwey