Skip to content

chore(doc): Temp workaround missing pytorch docs#577

Merged
cyx-6 merged 1 commit intoapache:mainfrom
junrushao:junrushao/2026-04-30/switch-pytorch-sphinx
Apr 30, 2026
Merged

chore(doc): Temp workaround missing pytorch docs#577
cyx-6 merged 1 commit intoapache:mainfrom
junrushao:junrushao/2026-04-30/switch-pytorch-sphinx

Conversation

@junrushao
Copy link
Copy Markdown
Member

@junrushao junrushao commented Apr 30, 2026

Will switch back once https://docs.pytorch.org/docs/stable/objects.inv becomes available again.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the PyTorch documentation URL in docs/conf.py from the stable version to a specific version. A review comment identifies a typo in the version number, noting that 2.11 should likely be 2.1, and suggests adding a TODO comment to revert this change once the stable documentation is fixed.

Comment thread docs/conf.py
"pillow": ("https://pillow.readthedocs.io/en/stable", None),
"numpy": ("https://numpy.org/doc/stable", None),
"torch": ("https://pytorch.org/docs/stable", None),
"torch": ("https://pytorch.org/docs/2.11", None),
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

The version 2.11 appears to be a typo for 2.1, as PyTorch does not have a version 2.11. Using an invalid version URL will prevent Intersphinx from correctly linking to PyTorch documentation. Additionally, since this is a temporary workaround, it is recommended to add a TODO comment to track its eventual reversal.

Suggested change
"torch": ("https://pytorch.org/docs/2.11", None),
"torch": ("https://pytorch.org/docs/2.1", None), # TODO: Revert to 'stable' once fixed

@cyx-6 cyx-6 merged commit 71d177b into apache:main Apr 30, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants