Skip to content

Conversation

fmassa
Copy link
Member

@fmassa fmassa commented Jul 31, 2019

Make it explicit that this was done with Kinetics400 in mind.

Also add docs.

cc @bjuncek

Copy link
Contributor

@bjuncek bjuncek left a comment

Choose a reason for hiding this comment

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

Looks good!

@codecov-io
Copy link

codecov-io commented Jul 31, 2019

Codecov Report

Merging #1183 into master will increase coverage by 0.06%.
The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1183      +/-   ##
==========================================
+ Coverage   65.82%   65.89%   +0.06%     
==========================================
  Files          79       79              
  Lines        5849     5849              
  Branches      890      890              
==========================================
+ Hits         3850     3854       +4     
+ Misses       1728     1726       -2     
+ Partials      271      269       -2
Impacted Files Coverage Δ
torchvision/datasets/__init__.py 100% <100%> (ø) ⬆️
torchvision/datasets/kinetics.py 34.78% <50%> (ø) ⬆️
torchvision/transforms/functional.py 71.38% <0%> (+0.28%) ⬆️
torchvision/transforms/transforms.py 81.53% <0%> (+0.58%) ⬆️

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 5c0b7f3...f3f7eea. Read the comment docs.

@fmassa fmassa merged commit fe4d17f into pytorch:master Jul 31, 2019
@fmassa fmassa deleted the kinetics-rename-docs branch July 31, 2019 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants