-
Notifications
You must be signed in to change notification settings - Fork 175
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
Automatic checkpointing #1667
base: master
Are you sure you want to change the base?
Automatic checkpointing #1667
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1667 +/- ##
===========================================
+ Coverage 50.50% 81.45% +30.95%
===========================================
Files 312 303 -9
Lines 19073 18476 -597
Branches 2795 2726 -69
===========================================
+ Hits 9632 15049 +5417
+ Misses 8733 2791 -5942
+ Partials 708 636 -72 ☔ View full report in Codecov by Sentry. |
486f2d1
to
a19d759
Compare
a37a9a0
to
7e375ff
Compare
6abe7d0
to
1284abe
Compare
1284abe
to
d72efb2
Compare
smallimpros remove prints fixes c ckp
(rebased it on master) |
Are you using this by any chance? I kind of abandoned the approach here because it had several issues. |
wanted to test it, hence the need for the rebase |
A long way to go...
Will need some help to make this play nice with sharding and MPI.
Must decide what to do with loggers. Right now we override the old ones. Can't really concatenate.
temporary interface.
if you run it twice, the second run goes from 50 to 100 steps