[Bugfix] Fix incorrect use of merge_size in Qwen3-VL video timestamp calculation #37439

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Isotr0py merged 5 commits into vllm-project:main from cnyvfang:main

Mar 18, 2026

+2 −2

vllm/model_executor/models/qwen3_vl.py

-Original file line number
+Diff line change
@@ Expand Up / @@ -767,7 +767,7 @@ def _get_video_second_idx( @@
             sampled_num_frames: int | None = None,
         ) -> list[int]:
             video_processor = self.get_video_processor()
-            merge_size = video_processor.merge_size
+            temporal_patch_size = video_processor.temporal_patch_size
             indices = metadata["frames_indices"]
             # metadata["fps"] refers to the true fps of the input video.
@@ Expand Down Expand Up / @@ -806,7 +806,7 @@ def _get_video_second_idx( @@
                     .astype(int)
                     .tolist()
                 )
-            timestamps = self._calculate_timestamps(indices, video_fps, merge_size)
+            timestamps = self._calculate_timestamps(indices, video_fps, temporal_patch_size)
             return timestamps
@@ Expand Down @@

Provide feedback