-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Update tensors and autograd #1185
Merged
Merged
Changes from 1 commit
Commits
Show all changes
17 commits
Select commit
Hold shift + click to select a range
e62ee7c
Update tensor_tutorial.py
subramen 0e867c7
Update autograd_tutorial.py
subramen e874967
Merge branch 'master' into 60mb_setup
brianjo edf2495
Fix import
subramen 6d094b7
Fix import
subramen cd6f97a
Fix import
subramen e0b21e3
Fixes issues in tensor, updates autograd
subramen bc50302
Adds "what is pytorch" to homepage
subramen 0d16b7b
Fixes formatting
subramen 6d91b98
Fix typo
subramen 75efe8b
Fixes typo
subramen adee684
fix failing test
subramen d7b0648
Merge branch 'master' into 60mb_setup
subramen 422296e
Merge branch 'master' into 60mb_setup
subramen 9dc3c79
Make suggested changes
subramen f56befe
Merge branch 'master' into 60mb_setup
subramen 9568cd2
Merge branch 'master' into 60mb_setup
subramen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Fix typo
- Loading branch information
commit 6d91b988d5d8c1d076f0cd500f9aff50f1e275e9
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you need a bit more text explanation here - either in text or in code comments.
very roughly something like:
"Let's start by looking at the a single training step. Here we load an existing pretrained resnet18 model, create random input data tensor of shape 3x64x64, and some random output labels."
...
Then we might want to pause again and say "here we push the data forward through the model, calculate the loss, and then call
.backward()
to collect the gradients for each parameter in the model. <more detail here on what backward does (i.e. where are the gradients stored?>....
Then we say: "Finally we load in an optimizer, in this case SGD, with a learning rate and momentum of ..., and call .step() to use our gradients and step backwards"
That's a basic model, below we dive a bit deeper into what is going on when we call .backward()
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree with Randall here