Turn off the new pipelining stuff by default; engines are too buggy.

[remoteglot] / remoteglot.pl

diff --git a/remoteglot.pl b/remoteglot.pl
index 9d88db75727817d0324f7c315e301e2f5b95fa22..28bebd25cfcf57de6e1685b83b202e9c03ebb70f 100755 (executable)
--- a/remoteglot.pl
+++ b/remoteglot.pl
@@ -23,6 +23,7 @@ my $target = "22";
 my $engine = "wine Rybkav2.3.2a.mp.w32.exe";
 my $telltarget = undef;   # undef to be silent
 my @tell_intervals = (5, 20, 60, 120, 240, 480, 960);  # after each move
+my $uci_assume_full_compliance = 0;                    # dangerous :-)
 
 # Program starts here
 $SIG{ALRM} = sub { output_screen(); };
@@ -60,7 +61,7 @@ while (<UCIREAD>) {
 }
 
 uciprint("setoption name UCI_AnalyseMode value true");
-uciprint("setoption name Preserve Analysis value true");
+# uciprint("setoption name Preserve Analysis value true");
 uciprint("setoption name NalimovPath value c:\\nalimov");
 uciprint("setoption name NalimovUsage value Rarely");
 uciprint("setoption name Hash value 1024");
@@ -116,8 +117,16 @@ while (1) {
                        # if we're already thinking on something, stop and wait for the engine
                        # to approve
                        if (defined($pos_calculating)) {
-                               uciprint("stop");
-                               $pos_waiting = $pos;
+                               if (!defined($pos_waiting)) {
+                                       uciprint("stop");
+                               }
+                               if ($uci_assume_full_compliance) {
+                                       $pos_waiting = $pos;
+                               } else {
+                                       uciprint("position fen " . $pos->{'fen'});
+                                       uciprint("go infinite");
+                                       $pos_calculating = $pos;
+                               }
                        } else {
                                # it's wrong just to give the FEN (the move history is useful,
                                # and per the UCI spec, we should really have sent "ucinewgame"),
@@ -161,7 +170,7 @@ sub handle_uci {
 
        chomp $line;
        $line =~ tr/\r//d;
-       print UCILOG "<= $line\n";
+       print UCILOG localtime() . " <= $line\n";
        if ($line =~ /^info/) {
                my (@infos) = split / /, $line;
                shift @infos;
@@ -174,7 +183,7 @@ sub handle_uci {
 
                parse_ids(@ids);
        }
-       if ($line =~ /^bestmove/) {
+       if ($line =~ /^bestmove/ && $uci_assume_full_compliance) {
                if (defined($pos_waiting)) {
                        uciprint("position fen " . $pos_waiting->{'fen'});
                        uciprint("go infinite");
@@ -838,7 +847,7 @@ sub can_reach {
 sub uciprint {
        my $msg = shift;
        print UCIWRITE "$msg\n";
-       print UCILOG "=> $msg\n";
+       print UCILOG localtime() . " => $msg\n";
 }
 
 sub short_score {