-
Notifications
You must be signed in to change notification settings - Fork 4.2k
Add tutorial for swap_tensors in nn.Module #2824
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
Conversation
A branch to test 2.3 tutorials
Disable usb_semisup_learn
* Update export tutorial --------- Co-authored-by: Svetlana Karslioglu <[email protected]>
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/tutorials/2824
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit 7dff2ed with merge base 2c231da ( This comment was automatically generated by Dr. CI and updates every 15 minutes. |
ae7bfca
to
05995f9
Compare
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.
Thanks so much @mikaylagawarecki! Looks great - I added a few editorial suggestions - feel free to add/ignore as needed.
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.
Very cool! Small suggestions only.
# Notably, the existing approach does not work in these cases: | ||
# | ||
# * when using ``__torch_dispatch__`` subclasses | ||
# * when ``param`` and ``new_param`` do not have the same type |
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 to define "type" here.
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.
Updated to say "Python type()
", do you think this is sufficiently expressive?
* [export] Remove interactive aoti compilation --------- Co-authored-by: Svetlana Karslioglu <[email protected]>
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.
Sounds good! Thanks!
as title