-
Notifications
You must be signed in to change notification settings - Fork 84
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
Update onnx.proto for int4 #3373
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #3373 +/- ##
========================================
Coverage 92.26% 92.26%
========================================
Files 500 500
Lines 20057 20057
========================================
Hits 18506 18506
Misses 1551 1551 ☔ View full report in Codecov by Sentry. |
Check results before merge 🔆 |
🔴bert_large_uncased_fp16: FAILED: MIGraphX is not within tolerance - check verbose output |
@lakhinderwalia please fix the format failure |
We should be ignoring this file since it's coming from onnx. |
Added an exception for this file name -- logically speaking this file should already be ignored by the existing rules for examining diffs. Not sure if this will actually fix the root cause here. |
Exception didnt work. @causten we should just merge it in. Since the format is done on a diff this wont affect other PRs. |
Update onnx.proto from https://github.com/onnx/onnx/blob/main/onnx/onnx.proto