Correct video_pad tag #2977
Merged
Correct video_pad tag #2977
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Chen, Peter <peter.chen@intel.com>
github-actions
bot
added
category: visual language
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR corrects a typo in the Qwen2-VL video pad tag documentation across multiple files. The tag <video_pad|> was missing the opening pipe character and has been corrected to <|video_pad|> to match the proper format used in other special tokens.
Key Changes:
- Fixed the malformed
<video_pad|>tag to<|video_pad|>in documentation strings - Applied the correction consistently across C++ headers, Python bindings, and type stub files
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/python/py_vlm_pipeline.cpp | Updated video_pad tag format in Python docstrings for VLM generate functions |
| src/python/openvino_genai/py_openvino_genai.pyi | Corrected video_pad tag in Python type stub documentation across multiple method signatures |
| src/cpp/include/openvino/genai/visual_language/pipeline.hpp | Fixed video_pad tag in C++ API documentation comments |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
xipingyan
approved these changes
Nov 6, 2025
Contributor
xipingyan
left a comment
xipingyan
left a comment
There was a problem hiding this comment.
Thanks， I copy them, and miss "|",
Wovchena
approved these changes
Nov 6, 2025
Wovchena
approved these changes
Nov 11, 2025
Merged
via the queue into
openvinotoolkit:master
with commit Nov 11, 2025
0fa4961
122 of 124 checks passed
krzyczar
pushed a commit
to krzyczar/openvino.genai
that referenced
this pull request
Nov 12, 2025
<!-- Keep your pull requests (PRs) as atomic as possible. That increases the likelihood that an individual PR won't be stuck because of adjacent problems, merge conflicts, or code review. Your merged PR is going to appear in the automatically generated release notes on GitHub. So the clearer the title the better. --> ## Description A trivial fixing of video_pad tag ## Checklist: - [ ] Tests have been updated or added to cover the new code. <!-- If the change isn't maintenance related, update the tests at https://github.com/openvinotoolkit/openvino.genai/tree/master/tests or explain in the description why the tests don't need an update. --> - [ ] This patch fully addresses the ticket. <!--- If follow-up pull requests are needed, specify in description. --> - [ ] I have made corresponding changes to the documentation. <!-- Run github.com/\<username>/openvino.genai/actions/workflows/deploy_gh_pages.yml on your fork with your branch as a parameter to deploy a test version with the updated content. Replace this comment with the link to the built docs. --> Signed-off-by: Chen, Peter <peter.chen@intel.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
A trivial fixing of video_pad tag
Checklist: