X-Git-Url: https://git.sesse.net/?p=stockfish;a=blobdiff_plain;f=src%2Fucioption.cpp;h=f671ecc5b07cb90261436dd80e77b89e2bc0c1a8;hp=d1326a4c9be0b8c76f1bd86f79c5faf3d720abc3;hb=5bb9da92872f298e6aedb59d728a42dc5719738e;hpb=455993b289b3c8f1c9c73af5876eb0190cde537c diff --git a/src/ucioption.cpp b/src/ucioption.cpp index d1326a4c..f671ecc5 100644 --- a/src/ucioption.cpp +++ b/src/ucioption.cpp @@ -106,8 +106,8 @@ namespace { o["Mate Threat Bonus"] = Option(3, 0, 8); o["Check Extension (PV nodes)"] = Option(2, 0, 2); o["Check Extension (non-PV nodes)"] = Option(1, 0, 2); - o["Single Reply Extension (PV nodes)"] = Option(2, 0, 2); - o["Single Reply Extension (non-PV nodes)"] = Option(2, 0, 2); + o["Single Evasion Extension (PV nodes)"] = Option(2, 0, 2); + o["Single Evasion Extension (non-PV nodes)"] = Option(2, 0, 2); o["Mate Threat Extension (PV nodes)"] = Option(0, 0, 2); o["Mate Threat Extension (non-PV nodes)"] = Option(0, 0, 2); o["Pawn Push to 7th Extension (PV nodes)"] = Option(1, 0, 2); @@ -116,15 +116,13 @@ 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["Full Depth Moves (PV nodes)"] = Option(10, 1, 100); - o["Full Depth Moves (non-PV nodes)"] = Option(3, 1, 100); - 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["Threads"] = Option(1, 1, 8); - o["Hash"] = Option(32, 4, 4096); + o["Threads"] = Option(1, 1, MAX_THREADS); + o["Hash"] = Option(32, 4, 8192); o["Clear Hash"] = Option(false, BUTTON); + o["New Game"] = Option(false, BUTTON); o["Ponder"] = Option(true); o["OwnBook"] = Option(true); o["MultiPV"] = Option(1, 1, 500); @@ -195,15 +193,18 @@ void init_uci_options() { load_defaults(options); - // Limit the default value of "Threads" to 7 even if we have 8 CPU cores. - // According to Ken Dail's tests, Glaurung plays much better with 7 than - // with 8 threads. This is weird, but it is probably difficult to find out - // why before I have a 8-core computer to experiment with myself. + // Set optimal value for parameter "Minimum Split Depth" + // according to number of available cores. assert(options.find("Threads") != options.end()); assert(options.find("Minimum Split Depth") != options.end()); - options["Threads"].defaultValue = stringify(Min(cpu_count(), 7)); - options["Threads"].currentValue = stringify(Min(cpu_count(), 7)); + Option& thr = options["Threads"]; + Option& msd = options["Minimum Split Depth"]; + + thr.defaultValue = thr.currentValue = stringify(cpu_count()); + + if (cpu_count() >= 8) + msd.defaultValue = msd.currentValue = stringify(7); } @@ -291,10 +292,28 @@ void set_option_value(const string& name, const string& value) { else if (v == "false") v = "0"; - if (options.find(name) != options.end()) - options[name].currentValue = v; - else + if (options.find(name) == options.end()) + { std::cout << "No such option: " << name << std::endl; + return; + } + + // Normally it's up to the GUI to check for option's limits, + // but we could receive the new value directly from the user + // by teminal window. So let's check the bounds anyway. + Option& opt = options[name]; + + if (opt.type == CHECK && v != "0" && v != "1") + return; + + else if (opt.type == SPIN) + { + int val = atoi(v.c_str()); + if (val < opt.minValue || val > opt.maxValue) + return; + } + + opt.currentValue = v; } @@ -314,7 +333,7 @@ void push_button(const string& buttonName) { bool button_was_pressed(const string& buttonName) { if (!get_option_value(buttonName)) - return false; + return false; set_option_value(buttonName, "false"); return true;