Skip to content

[EP ABI] Check if nodes specified in GetCapability() have already been assigned #26156

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.

Sign up for GitHub

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 13 commits into from
Sep 26, 2025
Merged

[EP ABI] Check if nodes specified in GetCapability() have already been assigned #26156

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
3e884f3
Check if nodes specified in GetCapability() have already been assigne…
adrianlizarraga Sep 25, 2025
0d8bdf7
Do not throw if only *some* nodes are already assigned to another EP
adrianlizarraga Sep 25, 2025
e24a908
Add unit tests
adrianlizarraga Sep 25, 2025
cdf1052
Update comment
adrianlizarraga Sep 25, 2025
17ea760
Add ORT_API_CALL
adrianlizarraga Sep 25, 2025
0010eab
Update onnxruntime/core/session/plugin_ep/ep_plugin_provider_interfac…
adrianlizarraga Sep 25, 2025
3d54e3d
Simplify testing and test more cases
adrianlizarraga Sep 25, 2025
03dcc93
Add warning for kSingleAssignNode
adrianlizarraga Sep 25, 2025
3dc9942
Don't use captured structured bindings feature (C++20 ext)
adrianlizarraga Sep 25, 2025
9a473dc
Address comments: check assigned EP in case for AddSingleNode
adrianlizarraga Sep 25, 2025
c7c7e59
When have to log an error, return immediately. When logging a warning…
adrianlizarraga Sep 25, 2025
369dcd3
Consistency: skip node grouping if any node is already assigned to an…
adrianlizarraga Sep 26, 2025
8782eb6
remove unused code now
adrianlizarraga Sep 26, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update onnxruntime/core/session/plugin_ep/ep_plugin_provider_interfac… 
…es.cc

Co-authored-by: Edward Chen <[email protected]>
  • Loading branch information
@adrianlizarraga @edgchen1
adrianlizarraga and edgchen1 authored Sep 25, 2025
commit 0010eabdee8cc08bfcfedab8f32759e561e0dd21
3 changes: 2 additions & 1 deletion onnxruntime/core/session/plugin_ep/ep_plugin_provider_interfaces.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,8 @@ static const Node* FindFirstNodeAssignedToOtherEP(const std::string& ep_type,
gsl::span<const EpNode* const> ep_nodes) {
auto node_iter = std::find_if(ep_nodes.begin(), ep_nodes.end(),
[&ep_type](const EpNode* node) -> bool {
return node->GetInternalNode().GetExecutionProviderType() != ep_type;
const auto& node_ep_type = node->GetInternalNode().GetExecutionProviderType();
return !node_ep_type.empty() && node_ep_type != ep_type;
});

return node_iter != ep_nodes.end() ? &(*node_iter)->GetInternalNode() : nullptr;
Expand Down
Loading