-
Notifications
You must be signed in to change notification settings - Fork 3.8k
Fix a few crashes while running shape inference #6195
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
Conversation
Signed-off-by: Xavier Dupre <xadupre@microsoft.com>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6195 +/- ##
==========================================
+ Coverage 56.95% 57.28% +0.32%
==========================================
Files 506 506
Lines 30467 31064 +597
Branches 4592 4619 +27
==========================================
+ Hits 17353 17795 +442
- Misses 12285 12446 +161
+ Partials 829 823 -6 ☔ View full report in Codecov by Sentry. |
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.
Thanks! I would recommend including the actual (invalid) values in all updated error messages applicable. I have found those to be the more helpful information in practice.
Signed-off-by: Xavier Dupre <xadupre@microsoft.com>
Signed-off-by: Liqun Fu <liqfu@microsoft.com>
…:convPoolShapeInference (#21507) ### Description onnx 1.16.2 is not available before ort 1.19.0 code freeze. Thus pick the needed change as patch
…:convPoolShapeInference (#21507) ### Description onnx 1.16.2 is not available before ort 1.19.0 code freeze. Thus pick the needed change as patch
Description
Fixes #6186, #6185, #6180, #6187.
It also improves the error messages to let users know where something is going wrong.