From 8b15961349e18a9ba113973c53f53913d0cd0fad Mon Sep 17 00:00:00 2001 From: joergoster Date: Wed, 3 May 2017 19:46:40 -0700 Subject: [PATCH] Fix multiPV issue #502 In general, this patch handles the cases where we don't have a valid score for each PV line in a multiPV search. This can happen if the search has been stopped in an unfortunate moment while still in the aspiration loop. The patch consists of two parts. Part 1: The new PVIdx was already part of the k-best pv's in the last iteration, and we therefore have a valid pv and score to output from the last iteration. This is taken care of with: bool updated = (i <= PVIdx && rootMoves[i].score != -VALUE_INFINITE); Case 2: The new PVIdx was NOT part of the k-best pv's in the last iteration, and we have no valid pv and score to output. Not from the current nor from the previous iteration. To avoid this, we are now also considering the previous score when sorting, so that the PV lines with no actual but with a valid previous score are pushed up again, and the previous score can be displayed. bool operator<(const RootMove& m) const { return m.score != score ? m.score < score : m.previousScore < previousScore; } // Descending sort I also added an assertion in UCI::value() to possibly catch similar issues earlier. No functional change. Closes #502 Closes #1074 --- src/search.cpp | 2 +- src/search.h | 3 ++- src/uci.cpp | 3 +++ 3 files changed, 6 insertions(+), 2 deletions(-) diff --git a/src/search.cpp b/src/search.cpp index ffe724aa..2c924014 100644 --- a/src/search.cpp +++ b/src/search.cpp @@ -1505,7 +1505,7 @@ string UCI::pv(const Position& pos, Depth depth, Value alpha, Value beta) { for (size_t i = 0; i < multiPV; ++i) { - bool updated = (i <= PVIdx); + bool updated = (i <= PVIdx && rootMoves[i].score != -VALUE_INFINITE); if (depth == ONE_PLY && !updated) continue; diff --git a/src/search.h b/src/search.h index ba8a9053..1218ef3b 100644 --- a/src/search.h +++ b/src/search.h @@ -57,7 +57,8 @@ struct RootMove { explicit RootMove(Move m) : pv(1, m) {} - bool operator<(const RootMove& m) const { return m.score < score; } // Descending sort + bool operator<(const RootMove& m) const { + return m.score != score ? m.score < score : m.previousScore < previousScore; } // Descending sort bool operator==(const Move& m) const { return pv[0] == m; } bool extract_ponder_from_tt(Position& pos); diff --git a/src/uci.cpp b/src/uci.cpp index 91a2260e..893c7e2f 100644 --- a/src/uci.cpp +++ b/src/uci.cpp @@ -18,6 +18,7 @@ along with this program. If not, see . */ +#include #include #include #include @@ -229,6 +230,8 @@ void UCI::loop(int argc, char* argv[]) { string UCI::value(Value v) { + assert(-VALUE_INFINITE < v && v < VALUE_INFINITE); + stringstream ss; if (abs(v) < VALUE_MATE - MAX_PLY) -- 2.39.2