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

Fix wrong sign for 200 TB score #5172

Closed

Disservin wants to merge 2 commits into official-stockfish:master from Disservin:fix-tb-200-issue

Member

Disservin commented Apr 13, 2024

TB values can have a distance of 0, mainly when we are in a tb position but haven't found mate.
Add a missing whitespace to UCIEngine::on_update_no_moves()

Reproduction:

./stockfish

setoption name SyzygyPath value /home/disservin/syzygy
position fen 1k6/8/8/8/8/8/2B1N3/3K4 w - - 0 1
go
info string NNUE evaluation using nn-ae6a388e4a1a.nnue
info string NNUE evaluation using nn-baff1ede1f90.nnue
info depth 1 seldepth 2 multipv 1 score cp -20000 nodes 17 nps 17000 hashfull 0 tbhits 17 time 1 pv c2g6

Disservin added 2 commits

April 13, 2024 21:54


          Fix wrong sign for 200 TB score

1f7534e


          Fix missing whitespace in UCIEngine::on_update_no_moves()

bd84561

vondele added no-functional-change to be merged labels

vondele closed this in

c55ae37

Disservin deleted the fix-tb-200-issue branch

April 13, 2024 20:26

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment