Removed an incorrect assert() statement in search.cpp, which asserted that

a static eval cached in the transposition table would always equal the static
eval of the current position. This is in general not true, because the cached
value could be from a previous search with different evaluation parameter
settings, or from a search from the opposite side (Stockfish's evaluation
function is assymmetric by default).

diff --git a/src/search.cpp b/src/search.cpp
index 695d4ae540ee432412de71c7791b564c0b717459..4449b1be0e2f4408bcb8b792b776221c4aa81517 100644 (file)
--- a/src/search.cpp
+++ b/src/search.cpp
@@ -1492,7 +1492,6 @@ namespace {
     else if (tte && tte->type() == VALUE_TYPE_EVAL)
     {
         // Use the cached evaluation score if possible
     else if (tte && tte->type() == VALUE_TYPE_EVAL)
     {
         // Use the cached evaluation score if possible

-        assert(tte->value() == evaluate(pos, ei, threadID));

         assert(ei.futilityMargin == Value(0));
 
         staticValue = tte->value();
         assert(ei.futilityMargin == Value(0));
 
         staticValue = tte->value();