Fix SAM2 window: use ALL frames when no kick detected

BUG: When kick_frame was None, SAM2 still only analyzed ~110 frames (4s)
instead of the full video.

FIX in _compute_sam_window_from_kick():
- If kick_frame is None → window = (0, total_frames) = ALL frames
- If kick_frame exists → window = 4 seconds around kick (unchanged)

FIX in _apply_selected_ball_to_yolo_state():
- Always call _compute_sam_window_from_kick(), even when kick is None
- Updated status message to indicate 'SAM2 will analyze ALL frames'

Added logging to show which window is being used.

app.py

@@ -288,16 +288,23 @@ def _compute_sam_window_from_kick(state: AppState, kick_frame: int | None) -> tu
     total_frames = state.num_frames
     if total_frames == 0:
         return 0, 0
-    fps = state.video_fps if state.video_fps and state.video_fps > 0 else 25.0
-    target_window_frames = max(1, int(round(fps * 4.0)))
-    half_window = target_window_frames // 2
+    
+    # If no kick detected, use ALL frames
     if kick_frame is None:
         start_idx = 0
+        end_idx = total_frames
+        print(f"[_compute_sam_window_from_kick] No kick detected → using ALL {total_frames} frames")
     else:
+        # If kick detected, use 4-second window around kick
+        fps = state.video_fps if state.video_fps and state.video_fps > 0 else 25.0
+        target_window_frames = max(1, int(round(fps * 4.0)))
+        half_window = target_window_frames // 2
         start_idx = max(0, int(kick_frame) - half_window)
-    end_idx = min(total_frames, start_idx + target_window_frames)
-    if end_idx <= start_idx:
-        end_idx = min(total_frames, start_idx + 1)
+        end_idx = min(total_frames, start_idx + target_window_frames)
+        if end_idx <= start_idx:
+            end_idx = min(total_frames, start_idx + 1)
+        print(f"[_compute_sam_window_from_kick] Kick @ {kick_frame} → window [{start_idx}, {end_idx}] ({end_idx - start_idx} frames)")
+    
     state.sam_window = (start_idx, end_idx)
     return start_idx, end_idx
 
@@ -1120,11 +1127,12 @@ def _apply_selected_ball_to_yolo_state(state: AppState) -> None:
         state.yolo_distance_from_start = distance_dict
         state.yolo_kick_distance = [distance_dict.get(f, 0.0) for f in frames_ordered]
     
-    # Update kick frame
+    # Update kick frame and SAM window
     kick_frame = tracking.get("kick_frame")
-    if kick_frame is not None:
-        state.kick_frame = kick_frame
-        _compute_sam_window_from_kick(state, kick_frame)
+    state.kick_frame = kick_frame  # Can be None
+    
+    # Always compute SAM window - if no kick, it will use ALL frames
+    _compute_sam_window_from_kick(state, kick_frame)
     
     # Mark as tracked
     state.is_yolo_tracked = True
@@ -1134,7 +1142,7 @@ def _apply_selected_ball_to_yolo_state(state: AppState) -> None:
     if kick_frame is not None:
         state.yolo_status = f"✅ Ball {idx+1} tracked. Kick @ frame {kick_frame}."
     else:
-        state.yolo_status = f"⚠️ Ball {idx+1} tracked ({coverage:.0%} coverage) but no kick detected."
+        state.yolo_status = f"⚠️ Ball {idx+1} tracked ({coverage:.0%} coverage) but no kick detected. SAM2 will analyze ALL frames."
 
 
 def draw_yolo_detections_on_frame(