owner

README.md

@@ -101,7 +101,7 @@ Keyboard shortcuts are shown with **[key]**.
     * **[w]**: All dots: Show all evaluation dots instead of the last few. 
         * You can configure how many are shown with this setting off, and whether they are shown for AIs under 'Play/Configure Teacher'.
     * **[e]**: Top moves: Show the next moves KataGo considered, colored by their expected point loss. Small dots indicate high uncertainty. Hover over any of them to see the principal variation.
-    * **[r]**: Show owner: Show expected ownership of each square.
+    * **[r]**: Show owner: Show expected ownership of each intersection.
     * **[t]**: NN Policy: Show KataGo's policy network evaluation, i.e. where it thinks the best next move is purely from the position, and in the absence of any 'reading'.
 
 * The analysis buttons are used for:
@@ -147,7 +147,6 @@ If you ever need to reset to the original settings, simply re-download the `conf
     * model: path to your KataGo model file.
     * config: path to your KataGo config file.    
     * threads: number of threads to use in the KataGo analysis engine.
-    * enable_ownership: calculate predicted ownership (slightly slower and more memory intensive).     
 * game settings
     * init_size: the initial size of the board, on start-up.
     * init_komi: likewise, for komi.

config.json

@@ -7,7 +7,7 @@
         "max_visits": 500,
         "fast_visits": 50,
         "max_time": 3.0,
-        "enable_ownership": true
+        "_enable_ownership": true
     },
     "sgf": {
         "sgf_load": "~/Downloads",

engine.py

@@ -130,7 +130,7 @@ class KataGoEngine:
         if next_move:
             moves.append(next_move)
         if ownership is None:
-            ownership = self.config["enable_ownership"] and not next_move
+            ownership = self.config["_enable_ownership"] and not next_move
         if visits is None:
             visits = self.config["max_visits"]
             if analyze_fast and self.config.get("fast_visits"):

gui/popups.py

@@ -108,7 +108,7 @@ class ConfigPopup(QuickConfigGui):
         for k1, all_d in sorted(self.config.items(), key=lambda tup: -len(tup[1])):  # sort to make greedy bin packing work better
             if k1 in self.ignore_cats:
                 continue
-            d = {k: v for k, v in all_d.items() if isinstance(v, (int, float, str, bool))}  # no lists . dict could be supported but hard to scale
+            d = {k: v for k, v in all_d.items() if isinstance(v, (int, float, str, bool)) and not k.startswith('_')}  # no lists . dict could be supported but hard to scale
             cat = GridLayout(cols=2, rows=len(d) + 1, size_hint=(1, len(d) + 1))
             cat.add_widget(Label(text=""))
             cat.add_widget(ScaledLightLabel(text=f"{k1} settings", bold=True))

katrain.kv

@@ -72,11 +72,11 @@
 
 <LabelledIntInput>:
     font_size: 0.5*self.size[1]
-    padding: 7, 0.1*self.size[1]
+    padding: 7, 0.2*self.size[1]
 
 <LabelledFloatInput>:
     font_size: 0.5*self.size[1]
-    padding: 7, 0.1*self.size[1]
+    padding: 7, 0.2*self.size[1]
 
 <LabelledTextArea>
     font_size: 0.2*self.size[1]