Do not re-run on tables opening/closing, address review.

examples/Model definition

@@ -389,10 +389,6 @@
               }
             }
           },
-          "position": {
-            "x": 667.0,
-            "y": 432.0
-          },
           "type": "basic"
         },
         "params": {
@@ -439,10 +435,6 @@
               }
             }
           },
-          "position": {
-            "x": 675.0,
-            "y": 499.0
-          },
           "type": "basic"
         },
         "params": {
@@ -496,10 +488,6 @@
               }
             }
           },
-          "position": {
-            "x": 1061.0,
-            "y": 239.0
-          },
           "type": "basic"
         },
         "params": {
@@ -554,10 +542,6 @@
             }
           },
           "params": {},
-          "position": {
-            "x": 1077.0,
-            "y": 334.0
-          },
           "type": "basic"
         },
         "params": {},
@@ -601,10 +585,14 @@
               }
             }
           },
-          "params": {},
-          "position": {
-            "x": 544.0,
-            "y": 306.0
+          "params": {
+            "name": {
+              "default": null,
+              "name": "name",
+              "type": {
+                "type": "None"
+              }
+            }
           },
           "type": "basic"
         },

lynxkite-app/src/lynxkite_app/crdt.py

@@ -91,6 +91,9 @@ def clean_input(ws_pyd):
         node.data.input_metadata = None
         node.data.error = None
         node.data.status = workspace.NodeStatus.done
+        for p in list(node.data.params):
+            if p.startswith("_"):
+                del node.data.params[p]
         node.position.x = 0
         node.position.y = 0
         if node.model_extra:

lynxkite-app/web/src/workspace/nodes/NodeWithTableView.tsx

@@ -15,11 +15,11 @@ function toMD(v: any): string {
   return JSON.stringify(v);
 }
 
+type OpenState = { [name: string]: boolean };
+
 export default function NodeWithTableView(props: any) {
   const reactFlow = useReactFlow();
-  const [open, setOpen] = useState(
-    props.data?.params?.tables_open ?? ({} as { [name: string]: boolean }),
-  );
+  const [open, setOpen] = useState((props.data?.params?._tables_open ?? {}) as OpenState);
   const display = props.data.display?.value;
   const single = display?.dataframes && Object.keys(display?.dataframes).length === 1;
   const dfs = Object.entries(display?.dataframes || {});
@@ -31,8 +31,11 @@ export default function NodeWithTableView(props: any) {
     }));
   }
   function toggleTable(name: string) {
-    setOpen((prevOpen) => ({ ...prevOpen, [name]: !prevOpen[name] }));
-    setParam("tables_open", { ...open, [name]: !open[name] });
+    setOpen((prevOpen: OpenState) => {
+      const newOpen = { ...prevOpen, [name]: !prevOpen[name] };
+      setParam("_tables_open", newOpen);
+      return newOpen;
+    });
   }
   return (
     <LynxKiteNode {...props}>

lynxkite-graph-analytics/src/lynxkite_graph_analytics/lynxkite_ops.py

@@ -291,7 +291,8 @@ def visualize_graph(
 
 
 @op("View tables", view="table_view")
-def view_tables(bundle: core.Bundle, *, tables_open: str = "", limit: int = 100):
+def view_tables(bundle: core.Bundle, *, _tables_open: str = "", limit: int = 100):
+    _tables_open = _tables_open  # The frontend uses this parameter to track which tables are open.
     return bundle.to_dict(limit=limit)