Skip to content

CVS-174486: [OV EP] Followup: Add tests for dynamic shapes and for empty initializers#956

Open
intbf wants to merge 3 commits intointel:ovep-developfrom
intbf:ext_init_more_unit_tests
Open

CVS-174486: [OV EP] Followup: Add tests for dynamic shapes and for empty initializers#956
intbf wants to merge 3 commits intointel:ovep-developfrom
intbf:ext_init_more_unit_tests

Conversation

@intbf
Copy link

@intbf intbf commented Mar 3, 2026

Introduce OVEP_ExtInit_DynamicEmbed_Tests to verify >2GB external weights handling and expected protobuf exception. Add OVEP_ExtInit_EmptyRawData_Tests to check correct backend behavior with empty raw_data initializers.

Relates to: https://jira.devtools.intel.com/browse/CVS-174486

Introduce OVEP_ExtInit_DynamicEmbed_Tests to verify >2GB external weights handling and expected protobuf exception. Add OVEP_ExtInit_EmptyRawData_Tests to check correct backend behavior with empty raw_data initializers.
Copy link

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 adds two new test fixture classes and corresponding test cases to openvino_ep_ext_init.cc to improve coverage of the OpenVINO EP's external initializer handling:

  1. OVEP_ExtInit_DynamicEmbed_Tests: Tests that creating a session with a model that has dynamic shapes and >2GB external weights (9 × 256MB = ~2.25GB) throws an Ort::Exception, since the OV EP must embed the weights into the protobuf which then exceeds protobuf's 2GB limit.

  2. OVEP_ExtInit_EmptyRawData_Tests: Tests that a model containing empty raw_data initializers (like empty tensors used as Resize inputs) is correctly handled by the OpenVINO EP backend, without crashing or producing incorrect output.

Changes:

  • Introduces OVEP_ExtInit_DynamicEmbed_Tests fixture with SetUpTestSuite/TearDownTestSuite and a test verifying that Ort::Exception is thrown for >2GB weight embedding
  • Introduces OVEP_ExtInit_EmptyRawData_Tests fixture with SetUpTestSuite/TearDownTestSuite and a test verifying correct inference output when empty raw_data initializers are present
  • Registers both new parameterized test suites for CPU, GPU, and NPU device variants

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

intbf and others added 2 commits March 4, 2026 04:29
Clean up comments. Refactor dynamic embed test to use local weights data, removing static member for better memory management. Add missing session config entry for consistency. Reformat exception test and increment offset in external data loop.
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.

3 participants