-
Notifications
You must be signed in to change notification settings - Fork 21
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
Dealing with invalid options.. #535
Comments
curretly we have this https://github.com/Disservin/fast-chess/blob/8115d51b94d80ce9104c4e2fe8722832765daaf4/app/src/engine/uci_engine.cpp#L173-#L175 but it doesnt work for some reason, possibly because stockfish doesnt stderr? |
must be a cutechess check based on the |
it only checks if the string was successfully send to the engine? there is no validation |
A (non-critical) TODO for |
should be fixed by the linked pr, should also have the same output syntax as cutechess |
when passing
option.FooBar=11
to an engine that doesn't have it, cutechess-cli will warn as in :If an existing option has an invalid value cutechess-cli will warn as in:
fishtest checks these warning messages:
https://github.com/vondele/fishtest/blob/bf62adf674e97ac69229aa879beb215c32d91ba6/worker/games.py#L1004-L1014
The text was updated successfully, but these errors were encountered: