Remove "Threat Depth" ucioption

This option likely has very low meaning for playing strength and style,
so I see no need to keep this configurable

No functional change

Signed-off-by: Marco Costalba <mcostalba@gmail.com>

diff --git a/src/search.cpp b/src/search.cpp
index f3e9c15d58776c8b79ae5e1565469763ab3bfd50..1bf56c8ea59a4ea8190f3e58b182c1ace6610a9e 100644 (file)
--- a/src/search.cpp
+++ b/src/search.cpp
@@ -171,6 +171,9 @@ namespace {
   // evaluation of the position is more than NullMoveMargin below beta.
   const Value NullMoveMargin = Value(0x200);
 
   // evaluation of the position is more than NullMoveMargin below beta.
   const Value NullMoveMargin = Value(0x200);
 

+  // Depth limit for use of dynamic threat detection when null move fails low
+  const Depth ThreatDepth = 5 * OnePly;
+

   // Step 9. Internal iterative deepening
 
   const Depth IIDDepthAtPVNodes = 5 * OnePly;
   // Step 9. Internal iterative deepening
 
   const Depth IIDDepthAtPVNodes = 5 * OnePly;


@@ -226,9 +229,6 @@ namespace {
 
   /// Variables initialized by UCI options
 
 
   /// Variables initialized by UCI options
 

-  // Depth limit for use of dynamic threat detection
-  Depth ThreatDepth;
-

   // Last seconds noise filtering (LSN)
   const bool UseLSNFiltering = true;
   const int LSNTime = 4000; // In milliseconds
   // Last seconds noise filtering (LSN)
   const bool UseLSNFiltering = true;
   const int LSNTime = 4000; // In milliseconds


@@ -429,8 +429,6 @@ bool think(const Position& pos, bool infinite, bool ponder, int side_to_move,
   MateThreatExtension[1] = Depth(get_option_value_int("Mate Threat Extension (PV nodes)"));
   MateThreatExtension[0] = Depth(get_option_value_int("Mate Threat Extension (non-PV nodes)"));
 
   MateThreatExtension[1] = Depth(get_option_value_int("Mate Threat Extension (PV nodes)"));
   MateThreatExtension[0] = Depth(get_option_value_int("Mate Threat Extension (non-PV nodes)"));
 

-  ThreatDepth   = get_option_value_int("Threat Depth") * OnePly;
-

   Chess960 = get_option_value_bool("UCI_Chess960");
   ShowCurrentLine = get_option_value_bool("UCI_ShowCurrLine");
   UseLogFile = get_option_value_bool("Use Search Log");
   Chess960 = get_option_value_bool("UCI_Chess960");
   ShowCurrentLine = get_option_value_bool("UCI_ShowCurrLine");
   UseLogFile = get_option_value_bool("Use Search Log");

diff --git a/src/ucioption.cpp b/src/ucioption.cpp
index bc35865540aa151b734b306b483d8e26a4110633..f671ecc5b07cb90261436dd80e77b89e2bc0c1a8 100644 (file)
--- a/src/ucioption.cpp
+++ b/src/ucioption.cpp
@@ -116,7 +116,6 @@ namespace {
     o["Passed Pawn Extension (non-PV nodes)"] = Option(0, 0, 2);
     o["Pawn Endgame Extension (PV nodes)"] = Option(2, 0, 2);
     o["Pawn Endgame Extension (non-PV nodes)"] = Option(2, 0, 2);
     o["Passed Pawn Extension (non-PV nodes)"] = Option(0, 0, 2);
     o["Pawn Endgame Extension (PV nodes)"] = Option(2, 0, 2);
     o["Pawn Endgame Extension (non-PV nodes)"] = Option(2, 0, 2);

-    o["Threat Depth"] = Option(5, 0, 100);

     o["Randomness"] = Option(0, 0, 10);
     o["Minimum Split Depth"] = Option(4, 4, 7);
     o["Maximum Number of Threads per Split Point"] = Option(5, 4, 8);
     o["Randomness"] = Option(0, 0, 10);
     o["Minimum Split Depth"] = Option(4, 4, 7);
     o["Maximum Number of Threads per Split Point"] = Option(5, 4, 8);