Skip to content

Register OpenVINO plugin for GenAI vision evaluation - #2748

Merged
Xiaoyu Z (xiaoyu-work) merged 2 commits into
mainfrom
aamancherla/ov-eval-ai2d
Oct 25, 2026
Merged

Xiaoyu Z (xiaoyu-work) merged 2 commits into
mainfrom
aamancherla/ov-eval-ai2d

Conversation

@ananyaa-ms

@ananyaa-ms ananyaa-ms commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Describe your changes

  • Register the onnxruntime-ep-openvino plugin for ONNX Runtime GenAI vision evaluation when the NPU target selects OpenVINOExecutionProvider, matching the existing QNN registration path.
  • Report missing-image and inference errors per sample instead of silently scoring blank predictions.
  • Add unit coverage for OpenVINO registration (single and split vision configurations), CPU behavior, and sample errors.

User-facing change: OpenVINO NPU GenAI vision models can be evaluated through Olive; individual sample failures now stop evaluation with a descriptive error.

Validation:

  • C:\Users\aamancherla\repos\olive-recipes\google-gemma-4-E2B-it\multi_comp\.venv\Scripts\python.exe -m pytest -q test\evaluator\test_olive_evaluator.py -k genai_vision --tb=short — passed (8 tests).
  • C:\Users\aamancherla\repos\olive-recipes\google-gemma-4-E2B-it\multi_comp\.venv\Scripts\python.exe -m ruff check --ignore CPY001 olive\evaluator\olive_evaluator.py test\evaluator\test_olive_evaluator.py — passed.
  • C:\Users\aamancherla\repos\olive-recipes\google-gemma-4-E2B-it\multi_comp\.venv\Scripts\python.exe -m ruff format --check olive\evaluator\olive_evaluator.py test\evaluator\test_olive_evaluator.py — passed (2 files).
  • Full test suite and lintrunner -a — not run locally.

Checklist before requesting a review

  • Add unit tests for this change.
  • Make sure all tests can pass.
  • Update documents if necessary.
  • Lint and apply fixes to your code by running lintrunner -a
  • Is this a user-facing change? If yes, give a description of this change to be included in the release notes.

(Optional) Issue link

Copilot AI balanced review requested due to automatic review settings October 20, 2026 00:06

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The focused changes have corresponding unit coverage, and no blocking issues were identified.

No open issues 🎉

What changed in this PR

Adds OpenVINO plugin registration to Olive’s GenAI vision evaluation for NPU targets and makes sample failures explicit.

Changes:

  • Registers OpenVINO once when selected for NPU evaluation.
  • Raises descriptive errors for missing images and inference failures.
  • Adds tests for single/split vision configurations, CPU behavior, and sample errors.
File Description
test/​evaluator/​test_olive_evaluator.py Tests OpenVINO registration and sample failure handling.
olive/​evaluator/​olive_evaluator.py Adds OpenVINO registration and replaces blank predictions on failure with errors.

Review effort: Balanced

Comment thread test/evaluator/test_olive_evaluator.py Fixed
Comment thread test/evaluator/test_olive_evaluator.py Fixed
Comment thread test/evaluator/test_olive_evaluator.py Fixed
Comment thread test/evaluator/test_olive_evaluator.py Fixed
Comment thread test/evaluator/test_olive_evaluator.py Fixed
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@xiaoyu-work
Xiaoyu Z (xiaoyu-work) merged commit 6701895 into main Oct 25, 2026
12 checks passed
@xiaoyu-work
Xiaoyu Z (xiaoyu-work) deleted the aamancherla/ov-eval-ai2d branch October 25, 2026 06:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants