Eric Firing wrote: > Ryan May wrote: >> Ok, >> >> I've committed my wind barbs stuff in SVN. Anyone interested, go >> ahead an hammer on it. > > Very nice! > > Comments on the example, barb_demo.py: > > 1) In your third panel, you put the args after the kwargs. I had no > idea this was even legal--it's the first time I have seen it. It's > probably better to stick with conventional usage so as not to distract > the reader from the point, which is how to use the barb function. > > 2) In a similar vein, I would suggest sticking with explicit X, Y, U, V > arguments. Your (*zip(*data)) is an obfuscation. > > 3) It would be better to use non-square asymmetric arrays so as to make > it clear which point in the data array corresponds to which point on the > plot. An example with NxN arrays doesn't help illustrate the relation > between the array and the plot locations. >
Ok, I checked in some changes that should address all of those, plus provide a more comprehensive demonstration of capabilities. Thanks for the feedback. I also got rid of the windows line endings (sorry). Usually not a problem, but I've had a (temporary) "opportunity" to be using windows for some development. Let me know if you see anything else. Ryan -- Ryan May Graduate Research Assistant School of Meteorology University of Oklahoma ------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ Matplotlib-devel mailing list Matplotlib-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/matplotlib-devel
