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

Add logic to resume ImageNet example with new learning rate #1118

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

kiersten-stokes
Copy link

Fixes #816

Resume no longer overwrites learning rate specific on command line.

(My linter also removed 2 extraneous spaces in a dict, I can revert this if desired 🙂)

@netlify
Copy link

netlify bot commented Feb 23, 2023

Deploy Preview for pytorch-examples-preview canceled.

Name Link
🔨 Latest commit adb0834
🔍 Latest deploy log https://app.netlify.com/sites/pytorch-examples-preview/deploys/63f8d9a99917f0000824719e

imagenet/main.py Outdated Show resolved Hide resolved
@kiersten-stokes
Copy link
Author

@soumith pinging for re-review 🙂

@kiersten-stokes
Copy link
Author

@soumith just pinging again to see if this is in a good state for approval 🙂

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.

Resume will override learning rate on command line.
3 participants