-
Notifications
You must be signed in to change notification settings - Fork 4k
[webgpu] Per-graph buffer manager for WebGPU multi-graph capture #28260
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 17 commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
94a2ca4
Per-graph buffer manager for WebGPU multi-graph capture
qjia7 bc5cfdb
Fix lintrunner trailing whitespace in factory.cc
qjia7 d24af03
Replace SetBufferManager with std::function getter and cached pointer…
qjia7 96ce10b
Make buffer manager getter nullable for static allocators
qjia7 93e3db6
Remove trailing blank line in constructor body
qjia7 cd24850
Address PR #28260 review comments
qjia7 9d65cd2
Verify output correctness and cleanup in TestReleaseCapturedGraph
qjia7 7417a7a
Improve TestReleaseCapturedGraph with two-op model and input variation
qjia7 ff5a405
Use three-op model (MatMul+Relu+MatMul) in TestReleaseCapturedGraph
qjia7 92f9fd7
Fix alignment of ReleaseCapturedGraphImpl parameter in ep.h
qjia7 2acc028
Fix lintrunner alignment in ep.h
qjia7 d9288df
Merge branch 'main' into per-graph-buffer-manager-webgpu
qjia7 09b39c9
Address PR review: fix API ordering, test isolation, and annotation I…
qjia7 f171fc8
Revert unrelated changes to io_binding_test.cc and ort_version_check.h
qjia7 09900dd
Restore Doxygen group closing marker in onnxruntime_c_api.h
qjia7 d8226af
Remove extra Doxygen group closing marker before SessionReleaseCaptur…
qjia7 f6db1b7
Fix version number in ReleaseCapturedGraph comment (26 -> 27)
qjia7 60f1ed1
Address edgchen1 PR review comments
qjia7 ea0e3a6
Address edgchen1 round 3 review: simplify allocator, use public APIs …
qjia7 086811b
Fix clang-format lint issues in factory.cc and webgpu_execution_provi…
qjia7 c805958
Add missing #include <numeric> for std::iota in graph_capture_test
qjia7 f9e5ffc
Combine graph annotation guards in OnRunStart and reuse iterator to a…
qjia7 b293976
Wrap BufferManager in lambda for plugin EP shared allocator in Create…
qjia7 0f61e15
Exclude graph capture test from plugin EP builds
qjia7 aa63475
Support graph capture test in both built-in and plugin WebGPU EP builds
qjia7 f5992e9
Fix undeclared ORT_UNUSED_PARAMETER in graph capture test
qjia7 d0369a5
Fix WebGPU EP MHA to ignore past key/value when no present outputs re…
Copilot e7cfaf0
Use generic wording in SessionReleaseCapturedGraph API doc
qjia7 bb9ebdd
Address PR review: add C++ API, use generic doc wording, fix line len…
qjia7 f0f3b07
Fix WebGPU plugin config key normalization in test provider helper
Copilot 3d9df67
Use global ort_env in graph capture test to avoid DLL unload
qjia7 673ed1f
Address review comments: mutex for ReleaseCapturedGraph, code style f…
qjia7 991c777
Make ReleaseCapturedGraph locking conditional to match Run() pattern
qjia7 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.