Skip to content

Conversation

fmassa
Copy link
Member

@fmassa fmassa commented May 21, 2019

This makes it consistent with the other models, which returns nouns in plurial

This makes it consistent with the other models, which returns nouns in plurial
@codecov-io
Copy link

codecov-io commented May 21, 2019

Codecov Report

Merging #934 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #934   +/-   ##
=======================================
  Coverage   60.26%   60.26%           
=======================================
  Files          63       63           
  Lines        5001     5001           
  Branches      745      745           
=======================================
  Hits         3014     3014           
  Misses       1784     1784           
  Partials      203      203
Impacted Files Coverage Δ
torchvision/models/detection/mask_rcnn.py 81.03% <ø> (ø) ⬆️
torchvision/models/detection/roi_heads.py 56.64% <100%> (ø) ⬆️
torchvision/models/detection/transform.py 78.09% <100%> (ø) ⬆️

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 cb99fe3...7fd07d8. Read the comment docs.

@fmassa fmassa merged commit 6e5599e into pytorch:master May 21, 2019
@fmassa fmassa deleted the rename-mask-masks branch May 21, 2019 12:49
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