Re: [matplotlib-devel] [patch] RegularPolyCollection.get_transformed_patches speed-up

2007-05-10 Thread Tim Leslie
Hi Eric, I just checked out your change and it works another 0.2s faster than what I had for my example, so I'm more than happy with the final result. Thanks for taking the time to put my hazy ideas into practice the right way :-) Cheers, Tim On 5/10/07, Eric Firing <[EMAIL PROTECTED]> wrote: >

Re: [matplotlib-devel] [patch] RegularPolyCollection.get_transformed_patches speed-up

2007-05-10 Thread Eric Firing
Tim, I couldn't resist; here is a numerix version of the method, which I committed to svn: def get_transformed_patches(self): # Shouldn't need all these calls to asarray; # the variables should be converted when stored. # Similar speedups with numerix should be at

Re: [matplotlib-devel] [patch] RegularPolyCollection.get_transformed_patches speed-up

2007-05-09 Thread Eric Firing
Tim Leslie wrote: > On 5/10/07, Eric Firing <[EMAIL PROTECTED]> wrote: >> Tim, >> >> Based on a *very superficial* quick look, I have two comments: >> >> 1) Since the plan is (or was?) to have one more release before >> specializing to numpy-only support, the ".T" won't work at present. >> >> 2) In

Re: [matplotlib-devel] [patch] RegularPolyCollection.get_transformed_patches speed-up

2007-05-09 Thread Tim Leslie
On 5/10/07, Eric Firing <[EMAIL PROTECTED]> wrote: > Tim, > > Based on a *very superficial* quick look, I have two comments: > > 1) Since the plan is (or was?) to have one more release before > specializing to numpy-only support, the ".T" won't work at present. > > 2) In the following (possibly man

Re: [matplotlib-devel] [patch] RegularPolyCollection.get_transformed_patches speed-up

2007-05-09 Thread Eric Firing
Tim, Based on a *very superficial* quick look, I have two comments: 1) Since the plan is (or was?) to have one more release before specializing to numpy-only support, the ".T" won't work at present. 2) In the following (possibly mangled by mailer), > +if Nsizes == 1: > +x

[matplotlib-devel] [patch] RegularPolyCollection.get_transformed_patches speed-up

2007-05-09 Thread Tim Leslie
Hi All, I've attached a patch which optimizes a particular case of the RegularPolyCollection.get_transformed_patches method. This is particularly useful when using a picker on a scatter plot with ~40,000 points (as I happen to be doing) and cuts the time spent in this function from ~4s to ~1s, wh