Skip to content

matplotlib.pyplot is missing from intersphinx #9331

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.

Sign up for GitHub

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

Jump to bottom
Closed
bsipocz opened this issue Oct 9, 2017 · 6 comments
Closed

matplotlib.pyplot is missing from intersphinx #9331

bsipocz opened this issue Oct 9, 2017 · 6 comments
Labels
Documentation
Milestone
v2.1.0-doc

Comments

@bsipocz
Copy link
Contributor

bsipocz commented Oct 9, 2017

The updates to the API docs for the new v2.1 release apparently has some consequences for downstream documentation.

In a few places in astropy we link to matplotlib.pyplot using intersphinx. However the new mapping doesn't have this it any more in the module mapping part.

I wonder what is the suggested approach? I would like to avoid hard wiring a link, or not linking at all. Also, the link using the old inventory resolves to http://matplotlib.org/api/pyplot_api.html#module-matplotlib.pyplot and it still seems to work, just missing from the new inventory.
@bsipocz bsipocz mentioned this issue Oct 9, 2017
Closed
@bsipocz bsipocz mentioned this issue Oct 10, 2017
Merged
@siebenkopf
Copy link

This is not only breaking your own links, but also links of other packages that want to use intersphinx to link to the matplotlib.pyplot module. For example, the following example sphinx documentation worked until recently, but does not work with version 2.1.0:

:py:mod:`matplotlib.pyplot`

It would be great if you could add back matplotlib.pyplot to the API documentation.

@anntzer
Copy link
Contributor

anntzer commented Oct 12, 2017

xref #8175 I guess the resolution should be similar.

@tacaswell
Copy link
Member

yikes, sorry about that!

@tacaswell tacaswell added this to the 2.1.0-docs milestone Oct 12, 2017
@tacaswell tacaswell mentioned this issue Oct 13, 2017
Merged
6 tasks
@tacaswell
Copy link
Member

The docs have been re-built and the xref should now be there.

Closed by #9386

@bsipocz
Copy link
Contributor Author

bsipocz commented Oct 13, 2017

Thank you @tacaswell!

@bsipocz bsipocz mentioned this issue Oct 13, 2017
Merged
@siebenkopf
Copy link

Great, thanks a lot :-)

@anntzer anntzer mentioned this issue Nov 16, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Documentation
Projects
None yet
Milestone
v2.1.0-doc
Development

No branches or pull requests
5 participants
@tacaswell @anntzer @siebenkopf @dstansby @bsipocz