MAINT: Change spectral to nipy_spectral, update docs, leave aliases#1127
MAINT: Change spectral to nipy_spectral, update docs, leave aliases#1127dmcdougall merged 3 commits intomatplotlib:masterfrom
Conversation
|
Check my docstring formatting. I used and for instance. not sure if that's right for functions |
|
I think you need double backquotes for the set_cmap call example, but that is completely from memory. |
lib/matplotlib/_cm.py
Outdated
There was a problem hiding this comment.
I know it's only aesthetic, but it wouldn't hurt to have all the data line up nicely.
There was a problem hiding this comment.
Oh I was thinking of refactoring the datad assignments into a single dictionary too. These sorts of OCD formatting are welcome? :)
There was a problem hiding this comment.
Personally, if it makes the code more readable then that's preferred. Ideally, @WeatherGod will weigh in with his PEP8 ninja chops :)
There was a problem hiding this comment.
Well PEP8 says not to align code for readability, but I think common usage disagrees. http://www.python.org/dev/peps/pep-0008/#pet-peeves
|
anything else that needs to be done here? |
|
@NelleV - your feedback on this would be highly valuable if you have a moment? |
|
LGTM 👍 |
Change spectral to nipy_spectral, update docs, leave aliases
endolith
changed the title
5 participants
For #879, rename
spectraltonipy_spectral(to distinguish fromSpectral). Leavespectralas an alias, andspectral()as a shortcut, but only in footnotes.