Dear Eric (and other developers),

I have implemented the requested changes and the resulting diff can be
seen at https://github.com/pschella/matplotlib/compare/master...cubehelix
I tried to do a better job at documentation and hope this is
sufficient, let me know if something is missing.
I'm not sure exactly what you meant with your suggestion to define
named functions inside the main function body but I hope my new
implementation more closely matches your wishes.
I have opted for a factory function within the main function to avoid
the code duplication associated with creating a named function for
each color component.

Kind regards,

Pim Schellart

P.S. Sorry for submitting the patch directly on the mailing list
before, I simply followed the link on how to contribute from the
matplotlib FAQ and didn't look at what was stated above the section
scrolled to automatically (maybe something to change).

------------------------------------------------------------------------------
BlackBerry® DevCon Americas, Oct. 18-20, San Francisco, CA
Learn about the latest advances in developing for the 
BlackBerry® mobile platform with sessions, labs & more.
See new tools and technologies. Register for BlackBerry® DevCon today!
http://p.sf.net/sfu/rim-devcon-copy1 
_______________________________________________
Matplotlib-devel mailing list
Matplotlib-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/matplotlib-devel

Reply via email to