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.

Eric

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