-
-
Notifications
You must be signed in to change notification settings - Fork 65
Pickle module #176
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
Pickle module #176
Conversation
✅ Deploy Preview for poutyne ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There are a few steps that failed/canceled in the CI/CD for formatting issues. Not sure if I am supposed to do something about that. |
Hi @atremblay , I'm currently on vacation so I'm going to look into your PR when I'm back in about two weeks. But yeah, if you can fix the formatting issue in the meantime, it would be great. |
@atremblay Could you fix the formatting so that we can go ahead with this? |
Ah think I figured it out. Was missing |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #176 +/- ##
=======================================
Coverage 94.20% 94.21%
=======================================
Files 46 46
Lines 3277 3282 +5
=======================================
+ Hits 3087 3092 +5
Misses 190 190 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Default behaviour of torch.load changed in pytorch 2.6. It used to have the argument
weights_only=False
, now it'sweights_only=True
.By specifying the
pickle_module
to use duringtorch.save
andtorch.load
, we can keep the new default value ofweights_only=True
.https://pytorch.org/docs/stable/notes/serialization.html#weights-only
pickle_module
exists at least since version 1.0.0 and the default value used to bepickle_module=pickle
.