[Bugfix] Fix unreachable None guard in Molmo2 get_candidate_target_fps (#55893)
Signed-off-by: shaolila <shaolila@buaa.edu.cn> Co-authored-by: shaolila <shaolila@buaa.edu.cn>
Adababy committed
Sep 9, 2026 at 20:37 UTC
94848eda600a07c28675f5753a11b2c212c146ed
1 file changed
+3
-2
vllm/multimodal/video.py
+3
-2
@@ -911,12 +911,13 @@ class Molmo2VideoBackend(VideoLoader):
911
ValueError: sampling_fps=2 must divide video_fps=5 to produce
912
consistent frame steps.
913
"""
914
+ if sampling_fps is None:
915
+ raise ValueError("sampling_fps must be provided")
916
+
917
video_fps = int(video_fps)
918
sampling_fps = int(sampling_fps)
919
max_fps = int(max_fps)
920
918
- if sampling_fps is None:
919
- raise ValueError("sampling_fps must be provided")
921
if video_fps <= 0 or sampling_fps <= 0:
922
raise ValueError(
923
"video_fps and sampling_fps must be positive "