From: Eric F. <ef...@ha...> - 2009-08-30 23:06:13
|
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 |