scipy: BUG: online doc returns 404 - wrong `reference` in url
The new theme seems to have introduced a duplicate reference
in the URL, e.g.,
- Before: https://docs.scipy.org/doc/scipy/reference/linalg.interpolative.html
- After: https://docs.scipy.org/doc/scipy/reference/reference/linalg.interpolative.html
The effect was that today all SciPy search engine results I found were broken.
The search engines will update eventually. But the double /reference/reference/
in the url is unnecessary, so it might be better removed?
About this issue
- Original URL
- State: closed
- Created 3 years ago
- Reactions: 5
- Comments: 56 (51 by maintainers)
Links to this issue
Commits related to this issue
- REL: fix doc upload path, remove "not yet released" line for 1.7.0 Closes gh-14267 [ci skip] — committed to rgommers/scipy by rgommers 3 years ago
- REL: fix doc upload path, remove "not yet released" line for 1.7.0 (#14272) Closes gh-14267 [ci skip] — committed to scipy/scipy by rgommers 3 years ago
- REL: fix doc upload path, remove "not yet released" line for 1.7.0 (#14272) Closes gh-14267 [ci skip] — committed to tylerjereddy/scipy by rgommers 3 years ago
- Use version suggested here: https://github.com/scipy/scipy/issues/14267#issuecomment-1003592442 — committed to tylerjereddy/htaccess_backup by tylerjereddy 2 years ago
- Update intersphinx reference for SciPy See https://github.com/scipy/scipy/issues/14267 and https://github.com/scipy/docs.scipy.org/pull/60. — committed to astrojuanlu/poliastro by astrojuanlu 2 years ago
- Update scipy intersphinx inventory for SciPy 1.8.0. According to https://github.com/scipy/scipy/issues/14267 the SciPy docs seems to have moved. — committed to hawkinsp/jax by hawkinsp 2 years ago
- Workaround scipy/scipy#14267 — committed to skirpichev/diofant by skirpichev 2 years ago
- Workaround scipy/scipy#14267 — committed to skirpichev/diofant by skirpichev 2 years ago
- Update intersphinx reference for SciPy See https://github.com/scipy/scipy/issues/14267 and https://github.com/scipy/docs.scipy.org/pull/60. — committed to astrojuanlu/poliastro by astrojuanlu 2 years ago
- REL: DOC: fix documentation URLs for the 1.8.x releases This is a forward port of gh-16221 Closes gh-14267 I applied this to the 1.8.0/1.8.1 docs, and things seem to work as expected after removing... — committed to rgommers/scipy by rgommers 2 years ago
- [skip ci] Fix scipy intersphinx link Follow-up from https://github.com/apache/tvm/pull/10181, as the URL has changed again. From [this comment](https://github.com/scipy/scipy/issues/14267#issuecomme... — committed to Lunderberg/tvm by Lunderberg 2 years ago
- [skip ci] Fix scipy intersphinx link Follow-up from https://github.com/apache/tvm/pull/10181, as the URL has changed again in https://github.com/scipy/scipy/pull/16221. From [this comment](https://g... — committed to Lunderberg/tvm by Lunderberg 2 years ago
- [skip ci] Fix scipy intersphinx link (#11399) Follow-up from https://github.com/apache/tvm/pull/10181, as the URL has changed again in https://github.com/scipy/scipy/pull/16221. From [this commen... — committed to apache/tvm by Lunderberg 2 years ago
- [skip ci] Fix scipy intersphinx link (#11399) Follow-up from https://github.com/apache/tvm/pull/10181, as the URL has changed again in https://github.com/scipy/scipy/pull/16221. From [this commen... — committed to juda/tvm by Lunderberg 2 years ago
It seems we’ve finally gotten this fixed in the release scripts now. The stable intersphinx link that I’d expect project to use is:
This has always worked, with short hiccups after the 1.8.0 and 1.8.1 releases. It shouldn’t be happening after the next release anymore.
This broke again after the 1.8.1 release. It should finally be fixed for good by:
If anyone still sees something that’s not in the correct place, please comment.
Updated the name so we hopefully have less new issues about this. We could also pin the issue?
Currently, it is difficult to reach the scipy docs from google:
https://user-images.githubusercontent.com/44469195/152681187-21be023a-3930-45e2-bfeb-417343881b19.mp4
The announcement is merged https://github.com/scipy/scipy.org/pull/396 but apparently the site didn’t get refreshed. It will show up soon. So I’ll close this since Ralf is probably going to see this
Thanks all for your considerations.
Some link redirects are now in place, at least temporarily per changes in https://github.com/tylerjereddy/htaccess_backup/pull/1
I guess we’ll see where the dust settles, and if we can live with URLs like https://docs.scipy.org/doc/scipy/html-scipyorg/ as the redirection target, or maybe can do something cleaner.
Likewise seeing 404 on links from google. For me only the ‘developer reference’ links pointing to http://scipy.github.io/devdocs/dev/ are working.
Concrete suggestions for improving
/srv/docs_scipy_org/doc/.htaccess
are welcome. Otherwise, I’ll probably just wait for some other core devs to make suggestions next week.Moving to 1.8.0, let’s not mix it with the release that introduces M1 support. Best not to upload any new docs for
1.7.3
(just like we didn’t for1.7.2
).I’ll fix that too, thanks for the suggestion.
Okay,
cp -R . ..
run on the server. That fixes the broken links to the API docs. I am changing theMakefile
and rebuilding, to push up a clean doc build and ensure we have the proper fix in place.The offending line is the last one here (from
doc/Makefile
in this repo):API docs are more important than tutorial though, so I’m tempted to drop the
reference/
anyway, it doesn’t quite make sense. Copying things over on the server to..
will give us double copies of everything, but seems like a good band aid to fix all links. Any concerns with doing that?@tupui is there any place where we can have a look at the doubling problem closely as it is picking up some traction? Maybe past related PRs where it might have been introduced