-
Notifications
You must be signed in to change notification settings - Fork 303
Switch to optimum-intel internal testing models #2919
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
base: master
Are you sure you want to change the base?
Switch to optimum-intel internal testing models #2919
Conversation
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 updates test configurations to use models from the optimum-intel-internal-testing organization instead of katuni4ka and other individual accounts. This consolidation improves test reliability and maintainability by using a centralized model repository for testing purposes.
Key changes:
- Migrated VLM (Vision-Language Model) test model references from
katuni4ka/*tooptimum-intel-internal-testing/* - Updated image generation model references to use the internal testing organization
- Standardized tokenizer test model references across the test suite
Reviewed Changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
tools/who_what_benchmark/tests/test_cli_vlm.py |
Updated VLM model reference to use optimum-intel-internal-testing organization |
tools/who_what_benchmark/tests/test_cli_image.py |
Migrated all image generation model references (SDXL, SD3, FLUX variants) to internal testing models |
tests/python_tests/test_vlm_pipeline.py |
Comprehensive update of VLM model IDs, including video models, across test parameters, fixtures, and configuration dictionaries |
tests/python_tests/test_tokenizer.py |
Updated tokenizer test models to use Phi3 and LLaVA models from internal testing organization |
tests/python_tests/test_structured_output.py |
Changed Phi3 model reference for structured output tests |
tests/python_tests/test_stateful_speculative_decoding.py |
Updated Gemma2 model reference for performance metrics test |
tests/python_tests/test_sampling.py |
Migrated Phi3 model references in sampling tests |
tests/python_tests/test_llm_pipeline.py |
Updated Phi3 and Gemma2 model references across multiple test functions |
tests/python_tests/samples/conftest.py |
Updated sample test configurations for MiniCPM, latent consistency, and LLaVA models |
tests/python_tests/data/tokenizer_configs.py |
Updated tokenizer configuration keys to match new model paths |
tests/python_tests/data/models.py |
Changed base model list to use internal testing organization |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
There's merge conflict |
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 11 out of 11 changed files in this pull request and generated no new 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 11 out of 11 changed files in this pull request and generated no new 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 13 out of 13 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Description
This PR updates tests to use models from the
optimum-intel-internal-testingorganization instead of individual HF accounts.Checklist: