Skip to content

Conversation

adrianlizarraga
Copy link
Contributor

@adrianlizarraga adrianlizarraga commented Jul 15, 2025

Description

  • Adds documentation to state that the data pointer for an OrtValue owned by an OrtGraph is stable during the lifetime of the OrtSession that owns the OrtGraph.
  • Adds documentation to the ort_graph_to_proto.h utils to show how to create a onnx::GraphProto with external initializers that actually point to in-memory data (same approach used internally within ORT).

Motivation and Context

Clarification of usage of new graph apis.

@adrianlizarraga adrianlizarraga changed the title [EP ABI] Document in-memory external initializers for ort_graph_to_proto util [EP ABI] Add documentation for OrtValue and ort_graph_to_proto util Jul 16, 2025
@adrianlizarraga adrianlizarraga marked this pull request as ready for review July 16, 2025 18:41
@adrianlizarraga adrianlizarraga merged commit af205c2 into main Jul 19, 2025
93 checks passed
@adrianlizarraga adrianlizarraga deleted the adrianl/ep-abi-proto-util-update-doc branch July 19, 2025 22:43
@snnn
Copy link
Member

snnn commented Jul 25, 2025

Hi there! We haven't cut the release branch for this version yet, so I'm removing the release:1.23.0 label for now to keep things tidy. Thanks so much for your contribution! We'll make sure this gets included when the release is prepared. 🤖

qti-yuduo pushed a commit to CodeLinaro/onnxruntime that referenced this pull request Aug 8, 2025
…icrosoft#25411)

### Description
- Adds documentation to state that the data pointer for an `OrtValue`
owned by an `OrtGraph` is stable during the lifetime of the `OrtSession`
that owns the `OrtGraph`.
- Adds documentation to the ort_graph_to_proto.h utils to show how to
create a `onnx::GraphProto` with external initializers that actually
point to in-memory data (same approach used internally within ORT).



### Motivation and Context
Clarification of usage of new graph apis.
sanketkaleoss pushed a commit to sanketkaleoss/onnxruntime that referenced this pull request Aug 11, 2025
…icrosoft#25411)

### Description
- Adds documentation to state that the data pointer for an `OrtValue`
owned by an `OrtGraph` is stable during the lifetime of the `OrtSession`
that owns the `OrtGraph`.
- Adds documentation to the ort_graph_to_proto.h utils to show how to
create a `onnx::GraphProto` with external initializers that actually
point to in-memory data (same approach used internally within ORT).



### Motivation and Context
Clarification of usage of new graph apis.
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.

5 participants