Skip to content

Fixed compute_clips docstring#1543

Merged
fmassa merged 1 commit into
pytorch:masterfrom
rsomani95:video-utils-docstring-typo
Oct 31, 2019
Merged

Fixed compute_clips docstring#1543
fmassa merged 1 commit into
pytorch:masterfrom
rsomani95:video-utils-docstring-typo

Conversation

@rsomani95

Copy link
Copy Markdown
Contributor

dilation was not passed into compute_clips or any of the functions called by it.

@codecov-io

codecov-io commented Oct 31, 2019

Copy link
Copy Markdown

Codecov Report

Merging #1543 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1543   +/-   ##
======================================
  Coverage    64.6%   64.6%           
======================================
  Files          87      87           
  Lines        6742    6742           
  Branches     1034    1034           
======================================
  Hits         4356    4356           
  Misses       2085    2085           
  Partials      301     301
Impacted Files Coverage Δ
torchvision/datasets/video_utils.py 72.25% <ø> (ø) ⬆️

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 ca57148...a6c838a. Read the comment docs.

@fmassa fmassa left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks

@fmassa fmassa merged commit b438d32 into pytorch:master Oct 31, 2019
@fmassa fmassa mentioned this pull request Oct 31, 2019
fmassa pushed a commit that referenced this pull request Oct 31, 2019
@rsomani95 rsomani95 deleted the video-utils-docstring-typo branch November 1, 2019 09:08
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