Update app.py

app.py

@@ -1333,7 +1333,6 @@ def render(d, show_labels_vals, path, mark_vals, mark_dist_vals, subsel, switchs
     return fig
 
 
-
 @app.callback(
     Output("path_info", "children"),
     Input("dim", "value"),
@@ -1346,8 +1345,10 @@ def path_info(d, path):
     if not path:
         return html.Span("Path: (empty)")
 
+    # Vertex list
     path_str = ", ".join(str(v) for v in path)
 
+    # Classification
     label, valid = classify_path(path, d)
     color = {
         "snake": "green",
@@ -1360,47 +1361,44 @@ def path_info(d, path):
     # Dimensions of consecutive edges
     dims = []
     for i in range(len(path) - 1):
-        dim = edge_dimension(path[i], path[i + 1])
-        dims.append(dim if dim is not None else "?")
+        ed = edge_dimension(path[i], path[i + 1])
+        dims.append(ed if ed is not None else "?")
     dims_str = ", ".join(str(x) for x in dims) if dims else "(none)"
 
-    extra = None
+    # If not snake, append violations inside the same bracket label
+    label_text = label
     if label == "not snake":
         viol = snake_violations(path, d)
 
         pairs = []
-        # Keep categories separate so it's interpretable
-        if viol["dup_vertices"]:
-            pairs.extend(viol["dup_vertices"])
-        if viol["non_adjacent_steps"]:
-            pairs.extend(viol["non_adjacent_steps"])
-        if viol["bad_closing_edge"]:
-            pairs.extend(viol["bad_closing_edge"])
-        if viol["chords"]:
-            pairs.extend(viol["chords"])
-
-        # Format: (a,b), (c,d), ...
-        # If there are many, don’t explode the UI
+        pairs.extend(viol.get("dup_vertices", []))
+        pairs.extend(viol.get("non_adjacent_steps", []))
+        pairs.extend(viol.get("bad_closing_edge", []))
+        pairs.extend(viol.get("chords", []))
+
+        # De-duplicate exact pairs (keep order)
+        seen = set()
+        uniq = []
+        for a, b in pairs:
+            key = (a, b)
+            if key not in seen:
+                seen.add(key)
+                uniq.append((a, b))
+
         MAX_SHOW = 30
-        shown = pairs[:MAX_SHOW]
+        shown = uniq[:MAX_SHOW]
         pairs_str = ", ".join(f"({a},{b})" for a, b in shown)
-        if len(pairs) > MAX_SHOW:
-            pairs_str += f", ... (+{len(pairs) - MAX_SHOW} more)"
+        if len(uniq) > MAX_SHOW:
+            pairs_str += f", ... (+{len(uniq) - MAX_SHOW} more)"
 
-        extra = html.Div(
-            [
-                html.Span("Violations: ", style={"fontWeight": "bold"}),
-                html.Span(pairs_str if pairs_str else "(none)", style={"fontFamily": "monospace"}),
-            ],
-            style={"marginTop": "4px"},
-        )
+        label_text = f"not snake. violations: {pairs_str if pairs_str else '(none)'}"
 
     return html.Div(
         [
             html.Div(
                 [
                     html.Span(f"Path: {path_str} "),
-                    html.Span(f"[{label}]", style={"color": color, "fontWeight": "bold"}),
+                    html.Span(f"[{label_text}]", style={"color": color, "fontWeight": "bold"}),
                 ]
             ),
             html.Div(
@@ -1409,10 +1407,10 @@ def path_info(d, path):
                     html.Span(dims_str, style={"fontFamily": "monospace"}),
                 ]
             ),
-            extra if extra is not None else html.Span(),
         ]
     )
 
+
 @app.callback(
     Output("path_bits", "children"),
     Input("dim", "value"),