Skip to content

fix: resolve the unexpected video frame drop issue of the InternVL model with multiple video inputs - #41121

Closed
WesKwong wants to merge 2 commits into
huggingface:mainfrom
WesKwong:main
Closed

WesKwong wants to merge 2 commits into
huggingface:mainfrom
WesKwong:main

Conversation

@WesKwong

Copy link
Copy Markdown

What does this PR do?

This PR addresses a bug where the InternVL preprocessor would incorrectly drop one video frame from subsequent videos in a multi-video input.

This issue was reported in: OpenGVLab/InternVL#1178

This commit addresses a bug where the InternVL preprocessor would incorrectly drop one video frame from subsequent videos in a multi-video input.

This issue was reported in: OpenGVLab/InternVL#1178
@Rocketknight1

Copy link
Copy Markdown
Member

cc @zucchini-nlp

@github-actions

Copy link
Copy Markdown
Contributor

[For maintainers] Suggested jobs to run (before merge)

run-slow: internvl

@WesKwong WesKwong changed the title Fix: Resolve unexpected video frame dropping for multi-video inputs fix: resolve the unexpected video frame drop issue of the InternVL model with multiple video inputs Sep 24, 2025
@ArthurZucker

Copy link
Copy Markdown
Collaborator

Thanks for the PR, and sorry it didn't make it in! This branch is based on v4, and v5 is very different, so rather than rebasing we'd love to invite you to open a new PR against main using our latest tools and good practices:

Closing this one in the meantime, thanks again!

@zucchini-nlp

zucchini-nlp commented Oct 21, 2026 •

Copy link
Copy Markdown
Member

I will check if issue is still real, and open a new PR if so.

Bug resolved atp

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants