-
Notifications
You must be signed in to change notification settings - Fork 4.2k
Remove references to TorchScript in docs #3453
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
gmagogsfm
commented
Jul 10, 2025
- Removes references to TorchScript
- Replace it with a deprecation warning and redirect to torch.export when appropriate
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/tutorials/3453
Note: Links to docs will display an error until the docs builds have been completed. This comment was automatically generated by Dr. CI and updates every 15 minutes. |
@@ -1,400 +0,0 @@ | |||
""" | |||
Introduction to TorchScript | |||
=========================== |
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.
For all deleted files, please add an entry to https://github.com/pytorch/tutorials/blob/main/redirects.py
>>> print(torch.ops.my_ops.warp_perspective) | ||
<built-in method custom::warp_perspective of PyCapsule object at 0x7ff51c5b7bd0> | ||
TorchScript is deprecated, please use | ||
`torch.export<https://docs.pytorch.org/tutorials/intermediate/torch_export_tutorial.html>`_ instead. |
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.
Please add an issue for all these pages with warning like this with the plan to delete them by {date} or {release}
Look like you have a build error: https://github.com/pytorch/tutorials/actions/runs/16257655203/job/45896761478?pr=3453 |
@svekars The warnings and redirect entries have been added. Can you pls review again? |
It looks good to me - thanks for creating the issue, too. |