Skip to content

Conversation

@mvafin
Copy link
Contributor

@mvafin mvafin commented Nov 4, 2025

Details:

  • Translate ONNX metadata with delegate

Tickets:

  • ticket-id

@mvafin mvafin requested a review from a team as a code owner November 4, 2025 13:58
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 implements ONNX metadata translation with delegate by adding metadata extraction and propagation functionality to the ONNX frontend. The changes enable the retrieval of metadata from ONNX models and its translation into OpenVINO's runtime information system.

  • Added virtual get_metadata() method to the GraphIterator interface
  • Implemented metadata extraction from ONNX protobuf models
  • Integrated metadata propagation into the translation session to set runtime info on the OpenVINO model

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/frontends/onnx/frontend/include/openvino/frontend/onnx/graph_iterator.hpp Adds virtual get_metadata() method to base GraphIterator interface
src/frontends/onnx/frontend/src/core/graph_iterator_proto.hpp Declares get_metadata() override in GraphIteratorProto class
src/frontends/onnx/frontend/src/core/graph_iterator_proto.cpp Implements metadata extraction from ONNX protobuf model properties
src/frontends/onnx/frontend/src/input_model.hpp Adds get_metadata() method declaration to InputModel
src/frontends/onnx/frontend/src/input_model.cpp Implements metadata storage and retrieval in InputModel implementation
src/frontends/onnx/frontend/src/translate_session.cpp Integrates metadata into OpenVINO model as runtime info under framework section
src/frontends/onnx/tests/graph_iterator.cpp Adds empty get_metadata() implementation to test SimpleIterator class

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

@github-actions github-actions bot added category: ONNX FE OpenVINO ONNX FrontEnd category: CPP API OpenVINO CPP API bindings labels Nov 4, 2025
@mvafin mvafin added this pull request to the merge queue Nov 5, 2025
Merged via the queue into openvinotoolkit:master with commit 90acd14 Nov 5, 2025
206 checks passed
@mvafin mvafin deleted the mvafin/onnx/metadata_delegate branch November 5, 2025 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: CPP API OpenVINO CPP API bindings category: ONNX FE OpenVINO ONNX FrontEnd

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants