Skip to content

Implement compiled model compatibility APIs in example plugin EP and add tests #27088

Merged
adrastogi merged 5 commits intomainfrom
adrastogi/add-compat-methods-to-ep-tests
Feb 6, 2026
Merged

Implement compiled model compatibility APIs in example plugin EP and add tests #27088
adrastogi merged 5 commits intomainfrom
adrastogi/add-compat-methods-to-ep-tests

Conversation

@adrastogi
Copy link
Contributor

Description

Add tests for the compiled model compatibility APIs (GetCompiledModelCompatibilityInfo and ValidateCompiledModelCompatibilityInfo) using the example plugin EP.

Changes:

  • Implement the compatibility methods in example_plugin_ep to enable testing
  • Update GetCapabilityImpl and CompileImpl to handle EPContext nodes (required for loading compiled models in tests)
  • Add PluginEp_CompatibilityInfo_WrittenToMetadata test - verifies compatibility info is written to compiled model metadata
  • Add PluginEp_CompatibilityInfo_ValidatedOnLoad test - verifies the round-trip: compile → load → validate compatibility

Motivation and Context

The compiled model compatibility APIs need test coverage to ensure they work correctly end-to-end. This exercises the full flow of writing compatibility metadata during compilation and validating it when loading a pre-compiled model. Additionally, by adding this support into the plugin EP, it enables us to add scenario tests that would exercise this in the context of Windows ML.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

You can commit the suggested changes from lintrunner.

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@adrastogi adrastogi marked this pull request as ready for review January 23, 2026 18:14
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

You can commit the suggested changes from lintrunner.

adrianlizarraga
adrianlizarraga previously approved these changes Feb 4, 2026
Copy link
Contributor

@adrianlizarraga adrianlizarraga left a comment

Choose a reason for hiding this comment

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

I think there are some linter issues remaining. Thank you for implementing the feedback.

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@adrastogi adrastogi merged commit 06fe9a4 into main Feb 6, 2026
90 checks passed
@adrastogi adrastogi deleted the adrastogi/add-compat-methods-to-ep-tests branch February 6, 2026 06:44
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.

2 participants