Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
Description
It's a bit confusing to have a proper module have such a short, non-descriptive name.
Looking at code that uses it, if one were to seefrom matplotlib import cm
- they'd be left wondering what those two letters could possibly stand for. Then further down, they'd see it being used with a keyword argument that would shed some light, such ascmap = cm.RdBu
, and they would maybe go through the steps of "oh, it's a map... probably a color map"...maybe...
If the import code was written asfrom matplotlib import colormaps as cm
, it'd be more clear. We would retain the danglingplt.cm
and have a dummymatplotlib.cm
module which would just import everything frommatplotlib.colormaps
.
as a bonus, I thinkplt.colormaps
should be retained, but the code and documentation for it should live inside this new colormaps module.