Skip to content

Fix redundant logic in video processing SmolVLM#45272

Merged
vasqu merged 3 commits intohuggingface:mainfrom
yonigozlan:nit-smolvlm-video
Apr 22, 2026
Merged

Fix redundant logic in video processing SmolVLM#45272
vasqu merged 3 commits intohuggingface:mainfrom
yonigozlan:nit-smolvlm-video

Conversation

@yonigozlan
Copy link
Copy Markdown
Member

@HuggingFaceDocBuilderDev
Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@yonigozlan
Copy link
Copy Markdown
Member Author

Cc @ydshieh When you have some time ;)

@yonigozlan yonigozlan requested a review from vasqu April 21, 2026 18:04
Copy link
Copy Markdown
Contributor

@vasqu vasqu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just sanity checking with run slow

@vasqu
Copy link
Copy Markdown
Contributor

vasqu commented Apr 22, 2026

run-slow: smolvlm

@github-actions
Copy link
Copy Markdown
Contributor

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

run-slow: smolvlm

@github-actions
Copy link
Copy Markdown
Contributor

Workflow Run ⚙️

This comment contains run-slow, running the specified jobs:

models: ["models/smolvlm"]
quantizations: []

@github-actions
Copy link
Copy Markdown
Contributor

CI Results

Workflow Run ⚙️

Commit Info

Context Commit Description
RUN 3a6b068a workflow commit (merge commit)
PR a956d25c branch commit (from PR)
main 0ea540ef base commit (on main)

✅ No failing test specific to this PR 🎉 👏 !

@vasqu vasqu added this pull request to the merge queue Apr 22, 2026
Merged via the queue into huggingface:main with commit 5617561 Apr 22, 2026
18 checks passed
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.

3 participants