-
Notifications
You must be signed in to change notification settings - Fork 13
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
Callbacks wishlist #5
Comments
Lines 564 to 570 in eada7e8
|
Hi @dfalbel, I have a couple callback ideas, is this issue a good place to share and brainstorm?? |
Sure @mattwarkentin ! |
I'm a big fan of
|
Thanks very much @mattwarkentin. I really like both callback ideas!
|
Great to hear! I have a working version of
|
Hmm, yeah, that one is tricky. FWIW it seems that lightning replaces the sampler for that. See https://github.com/PyTorchLightning/pytorch-lightning/blob/a64cc373946a755ce6c3aef57c1be607dfe29a0c/pytorch_lightning/trainer/data_loading.py#L169-L228 The way they are doing is similar to: mlverse/torch#621 (comment) ie, creates a new data loader with all the arguments equal to the first one except from the ones you want the change. |
The text was updated successfully, but these errors were encountered: