Skip to content
This repository has been archived by the owner on Jan 25, 2025. It is now read-only.

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 subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant