Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removed temporarily the MobileNetV3 Small quantized model endpoint #3366

Merged
merged 1 commit into from
Feb 9, 2021

Conversation

datumbox
Copy link
Contributor

@datumbox datumbox commented Feb 9, 2021

The smaller version of mobilenetv3 seems to have some problems that affect its accuracy. While we investigate we will remove the endpoint.

This PR needs to be reverted before picking up again the #3363.

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.

Thanks!

@codecov
Copy link

codecov bot commented Feb 9, 2021

Codecov Report

Merging #3366 (2578cef) into master (97885cb) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3366      +/-   ##
==========================================
- Coverage   74.81%   74.80%   -0.02%     
==========================================
  Files         105      105              
  Lines        9720     9716       -4     
  Branches     1561     1561              
==========================================
- Hits         7272     7268       -4     
  Misses       1961     1961              
  Partials      487      487              
Impacted Files Coverage Δ
torchvision/models/quantization/mobilenet.py 100.00% <100.00%> (ø)
torchvision/models/quantization/mobilenetv3.py 88.23% <100.00%> (-0.66%) ⬇️

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 97885cb...2578cef. Read the comment docs.

@datumbox datumbox merged commit 1b7c0f5 into pytorch:master Feb 9, 2021
@datumbox datumbox deleted the remove_mobilenetv3_small_quant branch February 9, 2021 16:33
facebook-github-bot pushed a commit that referenced this pull request Feb 10, 2021
Reviewed By: fmassa

Differential Revision: D26341412

fbshipit-source-id: 26147c8944588177cb20cebbadae8110e50e9dc3
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