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

uci: Add support for WDL in info lines #144

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mikeb26
Copy link
Contributor

@mikeb26 mikeb26 commented Nov 6, 2024

When UCI_ShowWDL is set, stockfish will additionally include win, loss, and draw probability values in its info evaluation lines. This commit augments uci.Info.UnmarshalText() to parse this information when available and store it within uci.Score.

When UCI_ShowWDL is set, stockfish will additionally include win,
loss, and draw probability values in its info evaluation lines. This
commit augments uci.Info.UnmarshalText() to parse this information when
available and store it within uci.Score.

(cherry picked from commit b83ccd9)
@mikeb26
Copy link
Contributor Author

mikeb26 commented Nov 26, 2024

@notnil go test is passing locally for me with this commit + stockfish 17. it looks like the automated test on github is still setup to use stockfish 14 which it can't find: https://github.com/notnil/chess/actions/runs/12035160790/job/33553454548?pr=144 any suggestions for how to proceed?

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