Skip to content

Conversation

@yatarkan
Copy link
Contributor

@yatarkan yatarkan commented Oct 27, 2025

Description

This PR updates tests to use models from the optimum-intel-internal-testing organization instead of individual HF accounts.

Checklist:

  • Tests have been updated or added to cover the new code
  • This patch fully addresses the ticket
  • I have made corresponding changes to the documentation - N/A

Copilot AI review requested due to automatic review settings October 27, 2025 11:22
@github-actions github-actions bot added category: WWB PR changes WWB category: visual language Visual language pipeline category: LLM LLM pipeline (stateful, static) category: sampling Sampling / Decoding algorithms category: tokenizers Tokenizer class or submodule update category: GGUF GGUF file reader labels Oct 27, 2025
Copy link
Contributor

Copilot AI left a 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/* to optimum-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.

@yatarkan yatarkan requested a review from Wovchena October 27, 2025 11:24
@Wovchena Wovchena enabled auto-merge October 28, 2025 07:36
@Wovchena
Copy link
Collaborator

There's merge conflict

Copilot AI review requested due to automatic review settings November 3, 2025 13:27
Copy link
Contributor

Copilot AI left a 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.

Copilot AI review requested due to automatic review settings November 25, 2025 20:38
Copy link
Contributor

Copilot AI left a 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.

Copilot AI review requested due to automatic review settings November 26, 2025 12:52
Copy link
Contributor

Copilot AI left a 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.

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

Labels

category: GGUF GGUF file reader category: LLM LLM pipeline (stateful, static) category: sampling Sampling / Decoding algorithms category: text streamer category: tokenizers Tokenizer class or submodule update category: visual language Visual language pipeline category: WWB PR changes WWB

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants