-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Update OnnxRuntime to latest version. #4337
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
OnnxRuntime supports win-x86, so start running tests on the win-x86 leg.
Codecov Report
@@ Coverage Diff @@
## master #4337 +/- ##
==========================================
- Coverage 74.58% 74.55% -0.03%
==========================================
Files 879 879
Lines 154524 154527 +3
Branches 16883 16882 -1
==========================================
- Hits 115249 115206 -43
- Misses 34526 34576 +50
+ Partials 4749 4745 -4
|
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.
🕐
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.
It looks like with this change our CI machines are bumping against the storage limit:
@chcosta - is there any way to request more storage on the hosted machines? I guess another option would be to clean up some space after we publish the test results, and before we build the NuGet packages. |
I cleaned up some space with #4338, which is allowing this PR to go through now. |
* Update OnnxRuntime to latest version. OnnxRuntime supports win-x86, so start running tests on the win-x86 leg. * Update README - ONNX is supported on 32-bit. * Consolidate "is onnx runtime supported" logic in the tests.
OnnxRuntime supports win-x86, so start running tests on the win-x86 leg.