Skip to content

[EP ABI] Update OrtGraph to use new OrtValues stored in internal Graph - #25159

Merged
adrianlizarraga merged 7 commits into
mainfrom
adrianl/EpAbi_UseGraphOrtValueInitializers
Jun 26, 2025
Merged

[EP ABI] Update OrtGraph to use new OrtValues stored in internal Graph#25159
adrianlizarraga merged 7 commits into
mainfrom
adrianl/EpAbi_UseGraphOrtValueInitializers

Conversation

@adrianlizarraga

@adrianlizarraga adrianlizarraga commented Jun 24, 2025

Copy link
Copy Markdown
Contributor

Description

Updates the OrtGraph implementation to take advantage of the work done in PR #23979, which sets the infrastructure to store initializers as OrtValue instances in the onnxruntime::Graph.

There still needs to be second part to the aforementioned PR to ensure that all initializers are stored as OrtValues in the Graph.

Motivation and Context

Comment thread onnxruntime/core/graph/ep_api_types.cc
Comment thread onnxruntime/core/graph/ep_api_types.cc
@adrianlizarraga
adrianlizarraga marked this pull request as ready for review June 25, 2025 17:22

@skottmckay skottmckay 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.

:shipit:

@adrianlizarraga
adrianlizarraga merged commit 47ddaaa into main Jun 26, 2025
155 of 157 checks passed
@adrianlizarraga
adrianlizarraga deleted the adrianl/EpAbi_UseGraphOrtValueInitializers branch June 26, 2025 19:41
ankus-qti pushed a commit to CodeLinaro/onnxruntime that referenced this pull request Nov 25, 2025
microsoft#25159)

### Description
Updates the `OrtGraph` implementation to take advantage of the work done
in PR microsoft#23979, which sets
the infrastructure to store initializers as `OrtValue` instances in the
`onnxruntime::Graph`.

There still needs to be second part to the [aforementioned
PR](microsoft#23979) to ensure that
all initializers are stored as `OrtValue`s in the Graph.



### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->
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