Skip to content
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

[ONNX] Upgrade ORT to fix CI #2579

Merged
merged 2 commits into from
Aug 14, 2020
Merged

Conversation

neginraoof
Copy link
Contributor

@neginraoof neginraoof commented Aug 12, 2020

Upgrade ORT to fix CI

Fixes #2571

@neginraoof
Copy link
Contributor Author

cc @fmassa for review. Thanks.

@codecov
Copy link

codecov bot commented Aug 13, 2020

Codecov Report

Merging #2579 into master will increase coverage by 1.83%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2579      +/-   ##
==========================================
+ Coverage   69.78%   71.61%   +1.83%     
==========================================
  Files          94       94              
  Lines        8118     8118              
  Branches     1289     1289              
==========================================
+ Hits         5665     5814     +149     
+ Misses       2039     1897     -142     
+ Partials      414      407       -7     
Impacted Files Coverage Δ
torchvision/io/video.py 69.82% <0.00%> (+1.18%) ⬆️
torchvision/models/detection/rpn.py 93.13% <0.00%> (+2.57%) ⬆️
torchvision/models/detection/roi_heads.py 82.34% <0.00%> (+5.10%) ⬆️
torchvision/__init__.py 78.12% <0.00%> (+6.25%) ⬆️
torchvision/datasets/video_utils.py 84.84% <0.00%> (+7.27%) ⬆️
torchvision/ops/poolers.py 97.05% <0.00%> (+11.76%) ⬆️
torchvision/ops/boxes.py 100.00% <0.00%> (+15.55%) ⬆️
torchvision/models/detection/transform.py 95.55% <0.00%> (+17.22%) ⬆️
torchvision/io/_video_opt.py 47.50% <0.00%> (+26.25%) ⬆️
torchvision/ops/_register_onnx_ops.py 78.94% <0.00%> (+28.94%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c547e5c...a73fbc0. Read the comment docs.

Copy link
Collaborator

@vfdev-5 vfdev-5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Travis is passing now 👍

@fmassa fmassa merged commit 27278ec into pytorch:master Aug 14, 2020
bryant1410 pushed a commit to bryant1410/vision-1 that referenced this pull request Nov 22, 2020
* Upgrade ORT to fix build

* Update .travis.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Travis CI is broken on onnx tests
3 participants