#4884: [with patch; needs review] Make colormap handling better
-------------------------+--------------------------------------------------
Reporter: abergeron | Owner: abergeron
Type: enhancement | Status: assigned
Priority: major | Milestone: sage-3.4
Component: graphics | Resolution:
Keywords: |
-------------------------+--------------------------------------------------
Changes (by abergeron):
* summary: [with patch; positive review] Make colormap handling better
=> [with patch; needs review] Make colormap
handling better
Comment:
To address was' comments (I hope), I killed the cmap_help() function, put
the appropriate info in all the relevant docstrings and added a mention
about how to obtain the colormap names.
I know you said you would prefer to include the list directly in the
docstring, but I don't think this would be productive as it would require
adding a doctest somewhere to check that the list has not changed and then
when that breaks, have the one responsible manually update all the
colormap lists, making sure to not forget every place where this info
could have been copy and pasted and to fix the doctest.
In short, it might be better, but I don't feel it is maintainable. Now if
there was some way to automatically include the current list in the
docstrings, then that is another story.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4884#comment:9>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---
