matplotlib icon indicating copy to clipboard operation
matplotlib copied to clipboard

[Doc]: Confusing page on color changes

Open timhoffm opened this issue 1 year ago • 4 comments

Documentation Link

https://matplotlib.org/devdocs/devel/color_changes.html

Problem

This page is a historic memo rather than documentation. But it doesn't say so.

Suggested improvement

IMHO this can be deleted. If somebody thinks, there is still relevant information, that's worth documenting, this needs a rework and an initial admonition for context and scope.

timhoffm avatar Jan 19 '24 12:01 timhoffm

It reads a bit like a MEP that was never made a formal MEP, so I'd move it there if we want to keep it.

story645 avatar Jan 19 '24 16:01 story645

If this should be kept we could also dump it into #14331 or create an issue link that in #14331.

But I'm still waiting for somebody to speak up for it.

timhoffm avatar Jan 22 '24 02:01 timhoffm

I'd get rid of it. Historically it exists in the old version of the docs, and I don't see what use it is.

jklymak avatar Jan 22 '24 03:01 jklymak

Looking through the history, looks like @tacaswell is the primary author so Tom do you have an opinion on what to do w/ this page?

story645 avatar Jan 22 '24 03:01 story645

Ping @tacaswell do you have an opinion?

timhoffm avatar Apr 07 '24 21:04 timhoffm

Reading through this again, this section https://matplotlib.org/devdocs/devel/color_changes.html#default-heat-map-colormap has a nice list of criteria for adding a new colormap that we may be want to bundle/rework into the coding guidelines or API guidelines as "criteria for new colormap" cause it comes up not infrequently?

story645 avatar Apr 10 '24 06:04 story645