[gnugo-devel] --experimental-break-in bugfixes

[Arend Bayer]

Two important bug fixes for --experimental-break-in.

Arend



Index: engine/breakin.c
===================================================================
RCS file: /cvsroot/gnugo/gnugo/engine/breakin.c,v
retrieving revision 1.1
diff -u -p -r1.1 breakin.c
--- engine/breakin.c    3 Jun 2003 00:13:11 -0000       1.1
+++ engine/breakin.c    4 Jun 2003 13:20:23 -0000
@@ -242,7 +242,7 @@ break_territories(int color_to_move, str
          size++;
       }
     if (size < 10)
-      return;
+      continue;

     if (color_to_move == OTHER_COLOR(territories.owner[k]))
       enlarge_goal(goal);
@@ -259,7 +259,9 @@ clear_break_in_list()
 /* The blocking moves should usually already have a move reason.
  *
  * The EXPAND_TERRITORY move reason ensures a territory evaluation of
- * this move, without setting the move.safety field.
+ * this move, without setting the move.safety field. (I.e. the move will
+ * be treated as a sacrifice move unless another move reasons tells us
+ * otherwise.)
  */
 void
 break_in_move_reasons(int color)
Index: engine/readconnect.c
===================================================================
RCS file: /cvsroot/gnugo/gnugo/engine/readconnect.c,v
retrieving revision 1.50
diff -u -p -r1.50 readconnect.c
--- engine/readconnect.c        4 Jun 2003 12:48:50 -0000       1.50
+++ engine/readconnect.c        4 Jun 2003 13:20:30 -0000
@@ -2946,6 +2946,9 @@ block_off(int str, char goal[BOARDMAX],
   int tactical_nodes;
   Hash_data goal_hash = goal_to_hashvalue(goal);

+  break_in_node_limit = connection_node_limit / 5;
+  break_in_depth = connect_depth2 - 5;
+
   if (move == NULL)
     move = &dummy_move;

Index: regression/connection.tst
===================================================================
RCS file: /cvsroot/gnugo/gnugo/regression/connection.tst,v
retrieving revision 1.58
diff -u -p -r1.58 connection.tst
--- regression/connection.tst   28 May 2003 00:10:55 -0000      1.58
+++ regression/connection.tst   4 Jun 2003 13:20:30 -0000
@@ -357,6 +357,15 @@ loadsgf games/nicklas/nicklas5.sgf 23
 105 disconnect F8 G6
 #? [1 G7]

+loadsgf games/nngs/Lazarus-gnugo-3.1.31-200204140254.sgf 164
+trymove black R14
+trymove white P16
+106 disconnect O14 O17
+#? [1 O16]
+popgo
+popgo
+
+
 # Report number of nodes visited by the tactical reading
 10000 get_reading_node_counter
 #? [0]&