-
Notifications
You must be signed in to change notification settings - Fork 571
Restrict ONNX opset to 16 and up #3051
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 32 commits
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
f56c4c9
Update ONNX-IR documentation with more comprehensive description
antimora 47339fa
Fix build issues with data structure changes
antimora e1557e9
Fix build issues with TensorType structure changes
antimora 1946252
Add static shape handling and rank inference for tensor operations
antimora 02d7b29
Fix clippy warnings
antimora 56de3f1
Merge remote-tracking branch 'upstream/main' into onnx-shape
antimora 731c6d9
Fix merge issues
antimora b23f3c6
Merge remote-tracking branch 'upstream/main' into onnx-shape
antimora d554562
Merge remote-tracking branch 'upstream/main' into onnx-shape
antimora f4e815c
Merge remote-tracking branch 'upstream/main' into onnx-shape
antimora d5acc51
Merge remote-tracking branch 'upstream/main' into onnx-shape
antimora 8728372
Enable unsqueeze with runtime axes values
antimora 55a677a
Fix clippy error
antimora 43af757
Remove default fall back
antimora 03cdbe5
Removed dead code.
antimora c9b32f2
Removed rank from TensroData
antimora 5ae4685
Removed elem_type from TensorData
antimora ee7f329
Merge remote-tracking branch 'upstream/main' into onnx-shape
antimora a29aba2
Merge remote-tracking branch 'upstream/main' into onnx-shape
antimora 1aeb4a1
Simplify elem_type match expressions with pattern grouping
antimora 285e361
Add static_shape back
antimora 31c7714
Add restriction for ONNX opset version >= 16
antimora dabeb29
Add onnx opset upgrade script
antimora b508df1
Update onnx-model.md
antimora 7693250
Removed onnx files for opsets < 16
antimora f6c2297
Skip opset upgrades if opset >= 16
antimora 768bb84
Bring back moved onnx file
antimora f607b89
Fix clippy
antimora 017fca3
Updated opset script per PR feedback
antimora bfa0388
Reimplement topk onnx and tests for opset16
antimora 662b57d
Merge branch 'main' into restrict-opset-16
antimora e15a2b3
Update README.md
antimora 1fd49aa
Include infer_shapes step in the upgrade script
antimora 759515b
Merge remote-tracking branch 'upstream/main' into restrict-opset-16
antimora 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
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
Binary file renamed
BIN
+581 Bytes
...t/onnx-tests/tests/clip/clip_opset16.onnx → ...rn-import/onnx-tests/tests/clip/clip.onnx
Binary file not shown.
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
Binary file not shown.
This file was deleted.
Oops, something went wrong.
Binary file renamed
BIN
+385 Bytes
...-tests/tests/dropout/dropout_opset16.onnx → ...ort/onnx-tests/tests/dropout/dropout.onnx
Binary file not shown.
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
Binary file not shown.
Binary file renamed
BIN
+529 Bytes
.../tests/reduce_sum/reduce_sum_opset13.onnx → ...nx-tests/tests/reduce_sum/reduce_sum.onnx
Binary file not shown.
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.
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.
Btw, why are there so many more steps in the
onnx_opset_upgrade.py
script? Actually, most of the additional code is for debug info or user input, but what about the additional shape inference?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.
Updated the document to include that step. I added it as a safe measure in case the older opset files didn't infer shapes already. But also as a precursor to use inferred node output shapes. For now, I think we can figure out Input/Output ranks from the static shapes.