Skip to content

Conversation

@ekagra-ranjan
Copy link
Contributor

Adds missing paper references to vgg and resnet variants.

Copy link
Member

@fmassa fmassa left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

Just waiting for CI to finish, and then this is good to merge!

@codecov-io
Copy link

codecov-io commented Jul 4, 2019

Codecov Report

Merging #1088 into master will decrease coverage by 0.12%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1088      +/-   ##
==========================================
- Coverage   64.66%   64.53%   -0.13%     
==========================================
  Files          68       68              
  Lines        5406     5406              
  Branches      829      829              
==========================================
- Hits         3496     3489       -7     
- Misses       1661     1665       +4     
- Partials      249      252       +3
Impacted Files Coverage Δ
torchvision/models/resnet.py 88.27% <ø> (ø) ⬆️
torchvision/models/vgg.py 89.55% <ø> (ø) ⬆️
torchvision/transforms/transforms.py 80.59% <0%> (-1.19%) ⬇️
torchvision/transforms/functional.py 71.09% <0%> (-0.29%) ⬇️

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 7d8cc19...4201e7c. Read the comment docs.

@fmassa fmassa merged commit 2eaeb2a into pytorch:master Jul 4, 2019
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.

3 participants