-
Notifications
You must be signed in to change notification settings - Fork 332
CVS-175484: Add MiniCPM-o-2_6 to precommit testing #3133
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
Conversation
…update requirements.txt
… with transformers==4.51.3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds MiniCPM-o-2_6 model support to precommit testing using the rkazants/tiny-random-MiniCPM-o-2_6 model. The model requires transformers version 4.51.3 or lower for export, so dedicated CI workflow steps have been added that downgrade transformers before running these specific tests. This is the first step toward comprehensive image resizing validation between OV GenAI and PIL.
Key changes:
- Added MiniCPM-o-2_6 model to test suite with appropriate configuration (image token format, size)
- Implemented version-specific skip conditions for incompatible transformers versions
- Created dedicated CI workflow steps across all platforms that downgrade transformers to 4.51.3 for MiniCPM-o-2_6 tests
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| tests/python_tests/test_vlm_pipeline.py | Added MiniCPM-o-2_6 model configuration, fixtures, test parameters, and transformers version check |
| tests/python_tests/requirements.txt | Added vocos and torchaudio dependencies required by MiniCPM-o-2_6 |
| .github/workflows/windows.yml | Added dedicated test step for MiniCPM-o-2_6 with transformers 4.51.3 |
| .github/workflows/manylinux_2_28.yml | Added dedicated test step for MiniCPM-o-2_6 with transformers 4.51.3 |
| .github/workflows/linux.yml | Added dedicated test step for MiniCPM-o-2_6 with transformers 4.51.3 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Vladimir Zlobin <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…l-internal-testing/tiny-random-MiniCPM-o-2_6
|
build_jenkins |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
There's merge conflict |
|
build_jenkins |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
build_jenkins |
…unused conversation logic
4c278c8 to
2c32f23
Compare
c1299d8
Description
Added MiniCPM-o-2_6 to precommit testing
This addresses the first part of the requirements in the CVS ticket. There will be further PR's to add precommit tests that actually resize the image within OV GenAI / optimum, and then compare results. This will require some changes to resize algorithms to better align them with PIL resize.
CVS-175484
Checklist: