Skip to content

Conversation

@qjia7
Copy link
Contributor

@qjia7 qjia7 commented Jul 11, 2025

This PR uses the existed RunOption gpu_graph_id to control whether to skip the graph capture. When the webgpu ep option enableGraphCapture is enabled, in RunOption, gpu_graph_id = -1 means skipping graph capture. Otherwise, go to the graph capture path for each session.run. If gpu_graph_id is not specified in RunOption, it will respect enableGraphCapture 's value to see whether to go to graph capture path.

@qjia7 qjia7 requested review from fs-eire and guschmue July 11, 2025 12:31
@guschmue guschmue added the ep:WebGPU ort-web webgpu provider label Jul 14, 2025
Copy link
Contributor

@fs-eire fs-eire left a comment

Choose a reason for hiding this comment

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

LGTM. In future we can probably support multiple records but for now the current implementation is OK.

@qjia7 qjia7 merged commit 1b584c1 into main Jul 27, 2025
93 of 96 checks passed
@qjia7 qjia7 deleted the per-run-capture branch July 27, 2025 00:57
sanketkaleoss pushed a commit to sanketkaleoss/onnxruntime that referenced this pull request Aug 11, 2025
This PR uses the existed RunOption `gpu_graph_id` to control whether to
skip the graph capture. When the webgpu ep option `enableGraphCapture`
is enabled, in RunOption, gpu_graph_id = -1 means skipping graph
capture. Otherwise, go to the graph capture path for each session.run.
If gpu_graph_id is not specified in RunOption, it will respect
`enableGraphCapture `'s value to see whether to go to graph capture
path.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ep:WebGPU ort-web webgpu provider

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants