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.

Magic!  I don't know either.  It's one of those things you stick in 
during development for expediency.

> 2) In a similar vein, I would suggest sticking with explicit X, Y, U, V 
> arguments.  Your (*zip(*data)) is an obfuscation.

Agreed.

> 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.

Sure.
I was going to tweak the demo anyways to show off more of the available 
parameters, so all of this will fit in well with that. I'll try to 
squeeze that in tonight.

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

Reply via email to