fix/upload-outputs-and-hough-shape

app.py

@@ -54,7 +54,7 @@ def on_upload(video_path):
     """New clip: size the slider, show frame 0, reset lines/frame state."""
     if not video_path:
         return (gr.update(maximum=1, value=0), None, "Upload a clip to begin.",
-                0, None, [], "")
+                0, None, [], "", [], 0)
     n, fps = probe_video(video_path)
     n_max = max(n - 1, 0)
     frame = grab_frame(video_path, 0)
@@ -367,4 +367,4 @@ with gr.Blocks(title="VAR Offside Visualizer") as demo:
 
 if __name__ == "__main__":
     demo.queue().launch(server_name="0.0.0.0", server_port=7860,
-                        theme=THEME, css=RF_CSS)
+                        theme=THEME, css=RF_CSS)

pipeline/autolines.py

@@ -60,7 +60,12 @@ def merge_lines(wmask):
                            minLineLength=70, maxLineGap=30)
     if segs is None:
         return []
-    segs = segs[:, 0]
+    # cv2.HoughLinesP is documented to return shape (N, 1, 4), but some
+    # OpenCV builds/wheels squeeze the middle axis and hand back (N, 4)
+    # directly. reshape(-1, 4) works for both layouts, whereas segs[:, 0]
+    # silently mis-indexes on the squeezed form (each "segment" ends up a
+    # bare scalar, which then fails to unpack in _seg_angle/_line_from_seg).
+    segs = segs.reshape(-1, 4)
     used = np.zeros(len(segs), bool)
     merged = []
     for i in range(len(segs)):
@@ -167,4 +172,4 @@ def propose_lines(frame_rgb):
     # button still reaches the other one.
     if len(fams) == 2:
         fams = [fams[1], fams[0]]
-    return fams
+    return fams