-
Notifications
You must be signed in to change notification settings - Fork 6.9k
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
Remove hardcoded dependency version for sphinx #3660
Conversation
@@ -2,7 +2,7 @@ | |||
# | |||
|
|||
# You can set these variables from the command line. | |||
SPHINXOPTS = -W # turn warnings into errors | |||
SPHINXOPTS = # -W # turn warnings into errors |
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.
The new sphinx version generates a bunch of new warnings. If that's OK I'll fix these in another PR to keep this one self contained.
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.
LGTM, thanks.
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 a ton!
Summary: * Try using the pyortch theme * directly embed script in layout.html instead * use more mordern script Reviewed By: NicolasHug Differential Revision: D28169165 fbshipit-source-id: 564c5babbc3a96fc32fc9ada891dc2bdd339eb74 Co-authored-by: Francisco Massa <fvsmassa@gmail.com>
Closes #3511
Towards #3652
This PR removes the old sphinx version pinning, and remove the use of the
sphinxcontrib-googleanalytics
(which was the reason why we had to pin sphinx).The new rendered docs properly embed the script in the header: https://510696-73328905-gh.circle-artifacts.com/0/docs/index.html
The solution was adapted from https://www.ericholscher.com/blog/2009/apr/5/adding-google-analytics-sphinx-docs/
Perhaps the pytorch theme could support this out of the box: pytorch/pytorch_sphinx_theme#109