Skip to content

Update weights of classification models to allow proper unpickling #3620

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

Merged
merged 3 commits into from
Mar 30, 2021

Conversation

NicolasHug
Copy link
Member

@NicolasHug NicolasHug commented Mar 30, 2021

EDIT: The title of this PR is wrong (my bad): it's detection models, not classification

Follow up to #3581

Fixes #2068 - weights are unchanged, but the files can now be properly unpickled.

The new pth files have been uploaded to the S3 bucket and to the FB internal db.

For ref, here's the script that was used: https://nbviewer.jupyter.org/gist/NicolasHug/fb2c28d758951959da94d3db326ce21b

@codecov
Copy link

codecov bot commented Mar 30, 2021

Codecov Report

Merging #3620 (4499b03) into master (66d777e) will not change coverage.
The diff coverage is n/a.

❗ Current head 4499b03 differs from pull request most recent head 924701b. Consider uploading reports for the commit 924701b to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3620   +/-   ##
=======================================
  Coverage   79.68%   79.68%           
=======================================
  Files         105      105           
  Lines        9822     9822           
  Branches     1582     1582           
=======================================
  Hits         7827     7827           
  Misses       1517     1517           
  Partials      478      478           
Impacted Files Coverage Δ
torchvision/models/alexnet.py 87.50% <ø> (ø)
torchvision/models/resnet.py 90.41% <ø> (ø)

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 968cefc...924701b. Read the comment docs.

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.

Changes LGTM if the weights are accessible to the users

@fmassa fmassa changed the title Update weights of detection models to allow proper unpickling Update weights of classification models to allow proper unpickling Mar 30, 2021
@fmassa fmassa merged commit 7d955df into pytorch:master Mar 30, 2021
facebook-github-bot pushed a commit that referenced this pull request Apr 13, 2021
…ckling (#3620)

Summary:
* Update URLS of detection models

* Empty commit after setting  read permission on S3

Reviewed By: NicolasHug

Differential Revision: D27706954

fbshipit-source-id: 3c560abe41688f7d3b2fa8792e8f157c89a4fb81

Co-authored-by: Francisco Massa <[email protected]>
@datumbox datumbox added bug and removed fix labels Apr 27, 2021
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.

Re-upload weights for old model with new serialization
4 participants