Skip to content

Conversation

@mitruska
Copy link
Contributor

@mitruska mitruska commented Dec 17, 2025

Description

This PR fixes two issues blocking usage of Template plugin:

  • Incorrect element type of the If op input (change f32 to bool)
  • Rope shape (accessing out of range dimension, get_shape on dynamic shape)

CVS-169296

Checklist:

  • Tested with models "Qwen/Qwen2.5-VL-3B-Instruct" and "optimum-intel-internal-testing/tiny-random-qwen2.5-vl"
  • Tests have been updated or added to cover the new code. Tests TBD
  • This patch fully addresses the ticket.

Copilot AI review requested due to automatic review settings December 17, 2025 09:27
@github-actions github-actions bot added the category: visual language Visual language pipeline label Dec 17, 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 fixes two critical issues preventing the Template plugin from working with Visual Language Models (VLM):

  • Corrects the element type of the If operation's condition input from f32 to boolean
  • Fixes Rope shape calculation by accessing the partial shape from the compiled model instead of attempting to get_shape on a dynamic tensor

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Wovchena Wovchena added this pull request to the merge queue Jan 14, 2026
Merged via the queue into openvinotoolkit:master with commit bc4b8b1 Jan 14, 2026
174 of 181 checks passed
github-merge-queue bot pushed a commit to openvinotoolkit/openvino that referenced this pull request Jan 20, 2026
### Details:
 - Enable VLM models on Template plugin:
    - Set shape and call shape infer for convolution with dynamic shapes
- Tested with models "Qwen/Qwen2.5-VL-3B-Instruct" and
"optimum-intel-internal-testing/tiny-random-qwen2.5-vl"
- Required fixes in openvino.genai:
openvinotoolkit/openvino.genai#3124


### Tickets:
 - 169296
github-merge-queue bot pushed a commit to openvinotoolkit/openvino that referenced this pull request Jan 20, 2026
### Details:
 - Enable VLM models on Template plugin:
    - Set shape and call shape infer for convolution with dynamic shapes
- Tested with models "Qwen/Qwen2.5-VL-3B-Instruct" and
"optimum-intel-internal-testing/tiny-random-qwen2.5-vl"
- Required fixes in openvino.genai:
openvinotoolkit/openvino.genai#3124


### Tickets:
 - 169296
github-merge-queue bot pushed a commit to openvinotoolkit/openvino that referenced this pull request Jan 20, 2026
### Details:
 - Enable VLM models on Template plugin:
    - Set shape and call shape infer for convolution with dynamic shapes
- Tested with models "Qwen/Qwen2.5-VL-3B-Instruct" and
"optimum-intel-internal-testing/tiny-random-qwen2.5-vl"
- Required fixes in openvino.genai:
openvinotoolkit/openvino.genai#3124


### Tickets:
 - 169296
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: visual language Visual language pipeline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants