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.
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
Subgraph extraction in ONNX models #4107
Subgraph extraction in ONNX models #4107
Changes from 9 commits
0446814
0c8a4f7
4a512fc
49d2908
15bcb02
fe7b3b1
7e70fda
53667fa
ddcffa3
0078926
b2373e4
b8fb3bd
45d11e1
2d3e298
f2fdf5e
48b3cd4
92addf2
59d7ef4
6eaee42
0e7822f
985ef97
263cd4b
1f18183
9972ac0
a39cbd0
4b86f01
ad7100c
2a3744b
6980948
8d5f739
b337772
6c67ec8
0ae591d
ee6831c
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Assuming that
node 5
is an operation and(in_A)
etc. are tensors, this documentation doesn't provide rationale why we are specifying node index, because there is only one consumer for each of the tensors on the schematics. In this case the tensor name should be enough to specify the cutting point.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As discussed offline, this will be addressed in a separate ticket: 47578