From 5b9cbeb4d870877c852e687901ef61cf3fbefdf4 Mon Sep 17 00:00:00 2001 From: AnonymDevOSS Date: Tue, 28 Oct 2025 13:55:14 +0100 Subject: [PATCH] fix: process video bug on unused else The loop never uses break, so the else clause always executes. However, since it iterates over a generator, the generator is already exhausted after the first pass... so the else block would iterate over nothing. So, i believe the 'else' is redundant. --- supervision/utils/video.py | 4 ---- 1 file changed, 4 deletions(-) diff --git a/supervision/utils/video.py b/supervision/utils/video.py index 3b281b4e2..ef9e603da 100644 --- a/supervision/utils/video.py +++ b/supervision/utils/video.py @@ -249,10 +249,6 @@ def callback(scene: np.ndarray, index: int) -> np.ndarray: ): result_frame = callback(frame, index) sink.write_frame(frame=result_frame) - else: - for index, frame in enumerate(video_frames_generator): - result_frame = callback(frame, index) - sink.write_frame(frame=result_frame) class FPSMonitor: