Skip to content
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

[skip ci] Fix scipy intersphinx link #11399

Merged
merged 1 commit into from
May 20, 2022

Conversation

Lunderberg
Copy link
Contributor

Follow-up from #10181, as the URL has changed again in scipy/scipy#16221. From this comment, the html-scipyorg portion wasn't intended to be part of the URL.

This should resolve the HTTP 404 occurring in Docs: GPU step (e.g. here), by accessing https://docs.scipy.org/doc/scipy-1.8.0/objects.inv
instead of https://docs.scipy.org/doc/scipy-1.8.0/html-scipyorg/objects.inv

Follow-up from apache#10181, as the URL
has changed again in scipy/scipy#16221.  From
[this
comment](scipy/scipy#14267 (comment)),
the `html-scipyorg` portion wasn't intended to be part of the URL.

This should resolve the HTTP 404 occurring in `Docs: GPU`
step (e.g. [here](https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-11269/13/pipeline/405#step-975-log-73)),
by accessing `https://docs.scipy.org/doc/scipy-1.8.0/objects.inv`
instead of
`https://docs.scipy.org/doc/scipy-1.8.0/html-scipyorg/objects.inv`
@masahi masahi merged commit 85e42b6 into apache:main May 20, 2022
@Lunderberg Lunderberg deleted the scipy_intersphinx_url branch May 20, 2022 20:11
@rgommers
Copy link

Hi, I think you want scipy': ('https://docs.scipy.org/doc/scipy', None), here. This is and always was the stable link (it was just broken for some days after the 1.8.0 release).

Lunderberg added a commit to Lunderberg/tvm that referenced this pull request May 23, 2022
…rsphinx link

Follow-up from apache#10181 and
apache#11399.  Thank you to @rgommers for
[pointing out]
(apache#11399 (comment))
that the non-versioned link is stable and working.  The use of
the versioned link was only introduced to work around the breakage of
the stable link, so this reverts to the pre-breakage behavior.
@Lunderberg
Copy link
Contributor Author

Thank you! I've opened a follow-up to update.

areusch pushed a commit that referenced this pull request May 23, 2022
…#11411)

Follow-up from #10181 and
#11399.  Thank you to @rgommers for
[pointing out]
(#11399 (comment))
that the non-versioned link is stable and working.  The use of
the versioned link was only introduced to work around the breakage of
the stable link, so this reverts to the pre-breakage behavior.
juda pushed a commit to juda/tvm that referenced this pull request Jun 21, 2022
Follow-up from apache#10181, as the URL
has changed again in scipy/scipy#16221.  From
[this
comment](scipy/scipy#14267 (comment)),
the `html-scipyorg` portion wasn't intended to be part of the URL.

This should resolve the HTTP 404 occurring in `Docs: GPU`
step (e.g. [here](https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-11269/13/pipeline/405#step-975-log-73)),
by accessing `https://docs.scipy.org/doc/scipy-1.8.0/objects.inv`
instead of
`https://docs.scipy.org/doc/scipy-1.8.0/html-scipyorg/objects.inv`
juda pushed a commit to juda/tvm that referenced this pull request Jun 21, 2022
…rsphinx link (apache#11411)

Follow-up from apache#10181 and
apache#11399.  Thank you to @rgommers for
[pointing out]
(apache#11399 (comment))
that the non-versioned link is stable and working.  The use of
the versioned link was only introduced to work around the breakage of
the stable link, so this reverts to the pre-breakage behavior.
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.

5 participants