Reinier Heeres wrote: > Eric, all, > > Here's the latest version of my color map patch. I deferred your > suggestion #4; perhaps we can fix that later. > > Please let me know what you think; if everybody is ok with it I'll > push to trunk. > > Regards, > Reinier
Looks good. I haven't tested it--I trust you have taken care of that. I suggest one tiny change, then go ahead and commit: In your change to cm.py, instead of importing and using the copy module, just do this: + datad[cmapname] = list(cmapspec) + datad[cmapname_r] = list(cmapspec) + datad[cmapname_r].reverse() Simpler, more readable, possibly faster (not that speed matters here). Thanks again. Eric ------------------------------------------------------------------------------ Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day trial. Simplify your report design, integration and deployment - and focus on what you do best, core application coding. Discover what's new with Crystal Reports now. http://p.sf.net/sfu/bobj-july _______________________________________________ Matplotlib-devel mailing list Matplotlib-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/matplotlib-devel
