Skip to content

[Doc]: colorbar documentation should be modernized #25541

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

Open
jklymak opened this issue Mar 24, 2023 · 7 comments
Open

[Doc]: colorbar documentation should be modernized #25541

jklymak opened this issue Mar 24, 2023 · 7 comments

Comments

@jklymak
Copy link
Member

jklymak commented Mar 24, 2023

Problem

Searching "Matplotlib colorbar" brings up lots of info from our website, but a lot of it is pretty old. Older ways for doing things, in particular the examples that use AxesDivider, should get crosslinks to the more standard idioms.

@Higgs32584
Copy link
Contributor

Higgs32584 commented Mar 24, 2023

Is there already sufficient, updated documentation that just needs to be more prominent in the search or do we need to rewrite some pages? we probably should get a list going of depreciated content

I know this is kind of off-topic, but maybe we should include dates when the last time a page/function was changed in the documentation just to give people an idea of how much they should trust it, cause I am unable to tell how recent some documentation is without parsing down the examples directory

@timhoffm
Copy link
Member

but maybe we should include dates when the last time a page/function was changed in the documentation just to give people an idea of how much they should trust it

I'm afraid this information is less useful than it sounds. There's ancient documentation that is still correct. OTOH a recent update could only fix a typo/link/small part of a page and does not guarantee all the page content is up-to-date.

@Higgs32584
Copy link
Contributor

I still think disclosing the lifespan of these things because as time goes on, the risk of failure heightens. Most news networks have articles that display "last updated", as to give readers references to when the last time someone looked it over, and made edits, or changes.

And "recent update" is still an update, and while it certainly does not guarantee that the page is 100% right, neither does having no dates at all. I believe it still gives us a convenient reference point as to when the last time someone touched a page was.

@jklymak
Copy link
Member Author

jklymak commented Mar 24, 2023

Is there already sufficient, updated documentation that just needs to be more prominent in the search or do we need to rewrite some pages? we probably should get a list going of depreciated content

I think that should audited. I think all the right documentation is somewhere, but maybe some more top-level should have a permanent home somewhere

For instance https://matplotlib.org/devdocs/users/explain/axes/colorbar_placement.html (which is newly in this location for 3.8) maybe should be rewritten as just Colorbars and be more general than just placing them. Or perhaps a page should be added to https://matplotlib.org/devdocs/users/explain/colors/. I think right now all we have that is general is https://matplotlib.org/stable/gallery/color/colorbar_basics.html which should probably be somewhere other than buried in the gallery.

@Higgs32584
Copy link
Contributor

I am also thinking here, as I was going through various examples, this page seems to not include import statements for various statements, I do not know if this is related, but a lot of these examples cannot be run by themselves. I assume that the standard is that each example can run without additional code, or with the expectation that you ran the previous notebook block before it.

It also says that you can download the full source code below, either as a .py or a .ipynb file, but neither properly formats so every line of code is commented/in a markdown file.

https://matplotlib.org/stable/tutorials/toolkits/axes_grid.html

Does this warrant opening up a separate issue? Or does this fall into this thread of outdated documentation? I found this when I was searching codex AxisDivider.

@rcomer
Copy link
Member

rcomer commented Mar 26, 2023

It also says that you can download the full source code below, either as a .py or a .ipynb file, but neither properly formats so every line of code is commented/in a markdown file.

Good spot! It looks like that particular tutorial is entirely written in the docstring. I would say that warrants its own issue.

@rcomer
Copy link
Member

rcomer commented Jan 1, 2024

@jklymak was this addressed by #26914?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants