Skip to content

Conversation

jcaip
Copy link
Contributor

@jcaip jcaip commented Sep 29, 2025

Fixes T237454985

Description

Realized that we have essentially two versions of the same tutorial, with the only difference being one is a jupyter notebook.

here and here for reference.

The PR removes the unstable_source rst tutorial and makes a couple of small updates of the jupyter notebook.

Checklist

  • The issue that is being fixed is referred in the description (see above "Fixes #ISSUE_NUMBER")
  • Only one issue is addressed in this pull request
  • Labels from the issue that this PR is fixing are added to this pull request
  • No unnecessary issues are included into this pull request.

Realized that we have essentially two versions of the same tutorial,
with the only difference being one is a jupyter notebook.

[here](https://docs.pytorch.org/tutorials/unstable/semi_structured_sparse.html) and [here](https://docs.pytorch.org/tutorials/advanced/semi_structured_sparse.html) for reference.

The PR removes the `unstable_source` rst tutorial and makes a couple of
small updates of the jupyter notebook.
Copy link

pytorch-bot bot commented Sep 29, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/tutorials/3599

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 42f4887 with merge base 6f79852 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@meta-cla meta-cla bot added the cla signed label Sep 29, 2025
@svekars svekars added tutorials_audit used on tutorial audit PRs pytorch_doc_audit labels Sep 29, 2025
Copy link
Contributor

@svekars svekars left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!
Can you add a redirect in redirects.py from unstable/semi_structured_sparse.html => advanced/semi_structured_sparse.html.

Also, please remove from unstble_source/unstable_index.rst:

:link: unstable/semi_structured_sparse.html

@jcaip jcaip merged commit f526f25 into main Sep 29, 2025
21 checks passed
jafraustro pushed a commit to jafraustro/tutorials that referenced this pull request Oct 1, 2025
Fixes [T237454985](https://www.internalfb.com/intern/tasks/?t=237454985)

## Description
Realized that we have essentially two versions of the same tutorial,
with the only difference being one is a jupyter notebook.


[here](https://docs.pytorch.org/tutorials/unstable/semi_structured_sparse.html)
and
[here](https://docs.pytorch.org/tutorials/advanced/semi_structured_sparse.html)
for reference.

The PR removes the `unstable_source` rst tutorial and makes a couple of
small updates of the jupyter notebook.
## Checklist
<!--- Make sure to add `x` to all items in the following checklist: -->
- [x] The issue that is being fixed is referred in the description (see
above "Fixes #ISSUE_NUMBER")
- [x] Only one issue is addressed in this pull request
- [x] Labels from the issue that this PR is fixing are added to this
pull request
- [x] No unnecessary issues are included into this pull request.
jafraustro pushed a commit to jafraustro/tutorials that referenced this pull request Oct 1, 2025
Fixes [T237454985](https://www.internalfb.com/intern/tasks/?t=237454985)

## Description
Realized that we have essentially two versions of the same tutorial,
with the only difference being one is a jupyter notebook.


[here](https://docs.pytorch.org/tutorials/unstable/semi_structured_sparse.html)
and
[here](https://docs.pytorch.org/tutorials/advanced/semi_structured_sparse.html)
for reference.

The PR removes the `unstable_source` rst tutorial and makes a couple of
small updates of the jupyter notebook.
## Checklist
<!--- Make sure to add `x` to all items in the following checklist: -->
- [x] The issue that is being fixed is referred in the description (see
above "Fixes #ISSUE_NUMBER")
- [x] Only one issue is addressed in this pull request
- [x] Labels from the issue that this PR is fixing are added to this
pull request
- [x] No unnecessary issues are included into this pull request.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants