Fix overzealous marking of dame

online-go · Jun 3, 2024 · baf11e9 · baf11e9
1 parent a5ecb78
commit baf11e9
Showing 1 changed file with 2 additions and 15 deletions.
diff --git a/src/autoscore.ts b/src/autoscore.ts
@@ -190,17 +190,13 @@ export function autoscore(
                     isBlack(white_plays_first_ownership[y][x])
                 ) {
                     remove(x, y, "both players agree this is captured by black");
-                }
-
-                if (
+                } else if (
                     board[y][x] === JGOFNumericPlayerColor.BLACK &&
                     isWhite(black_plays_first_ownership[y][x]) &&
                     isWhite(white_plays_first_ownership[y][x])
                 ) {
                     remove(x, y, "both players agree this is captured by white");
-                }
-
-                if (
+                } else if (
                     board[y][x] === JGOFNumericPlayerColor.EMPTY &&
                     isDameOrUnknown(black_plays_first_ownership[y][x]) &&
                     isDameOrUnknown(white_plays_first_ownership[y][x])
@@ -236,15 +232,6 @@ export function autoscore(
                     is_settled[y][x] = 1;
                     settled[y][x] = JGOFNumericPlayerColor.BLACK;
                 }
-
-                if (
-                    isDameOrUnknown(black_plays_first_ownership[y][x]) &&
-                    isDameOrUnknown(white_plays_first_ownership[y][x])
-                ) {
-                    is_settled[y][x] = 1;
-                    settled[y][x] = JGOFNumericPlayerColor.EMPTY;
-                    remove(x, y, "both players agree this is dame");
-                }
             }
         }