Skip to content

Conversation

fmassa
Copy link
Member

@fmassa fmassa commented Aug 1, 2019

I had do modify the docstrings to use Napoleon NumPy style, because Napoleon Google Style doesn't support multiple return arguments.

Had do modify the docstrings to use Napoleon NumPy style, because Napoleon Google Style doesn't support multiple return arguments
@fmassa fmassa requested a review from soumith August 1, 2019 12:05
@codecov-io
Copy link

codecov-io commented Aug 1, 2019

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #1189     +/-   ##
=========================================
+ Coverage   65.64%   65.74%   +0.1%     
=========================================
  Files          79       79             
  Lines        5827     5827             
  Branches      889      889             
=========================================
+ Hits         3825     3831      +6     
+ Misses       1731     1728      -3     
+ Partials      271      268      -3
Impacted Files Coverage Δ
torchvision/io/video.py 72% <ø> (ø) ⬆️
torchvision/ops/boxes.py 94.73% <ø> (ø) ⬆️
torchvision/transforms/transforms.py 81.13% <0%> (+0.19%) ⬆️
torchvision/transforms/functional.py 71.38% <0%> (+1.44%) ⬆️

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 9168476...613d610. Read the comment docs.

@fmassa fmassa merged commit 4ec38d4 into pytorch:master Aug 2, 2019
@fmassa fmassa deleted the doc-io-ops branch August 2, 2019 15:11
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.

2 participants