Re: [matplotlib-devel] Patch for scatter plot legend enhancement

2008-10-08 Thread Erik Tollerud
Ah, that makes more sense Jae-Joon - thanks! With this addition it all looks fine to me - I've attached a patch against 6174 that does everything I'd like to see it do... note that I've left in the TODO in collections.py simply because I still am not certain that what I'm doing there is correct in

Re: [matplotlib-devel] path simplification with nan (or move_to)

2008-10-08 Thread John Hunter
On Wed, Oct 8, 2008 at 8:40 PM, Eric Firing <[EMAIL PROTECTED]> wrote: > Thanks for doing this--it has already helped me in my testing of the > gappy-path simplification support, which I have now committed. As you > suggested earlier, I included in path.py a check for a compatible codes > array. >

Re: [matplotlib-devel] path simplification with nan (or move_to)

2008-10-08 Thread Eric Firing
Michael Droettboom wrote: > John Hunter wrote: >> On Wed, Oct 8, 2008 at 11:37 AM, Michael Droettboom <[EMAIL PROTECTED]> >> wrote: >> >> >>> I figured this out. When this happens, a RuntimeError("Agg rendering >>> complexity exceeded") is thrown. >>> >> Do you think it is a good idea to

Re: [matplotlib-devel] Bug in legend

2008-10-08 Thread John Hunter
On Wed, Oct 8, 2008 at 1:44 PM, David Huard <[EMAIL PROTECTED]> wrote: > /usr/local/lib64/python2.5/site-packages/matplotlib/legend.pyc in > __init__(self, parent, handles, labels, loc, numpoints, prop, pad, > borderpad, markerscale, labelsep, handlelen, handletextsep, axespad, shadow) > 125

[matplotlib-devel] Bug in legend

2008-10-08 Thread David Huard
I just updated matplotlib from svn and here is traceback I get after calling legend with the pad argument: /usr/local/lib64/python2.5/site-packages/matplotlib/pyplot.pyc in legend(*args, **kwargs) 2390 def legend(*args, **kwargs): 2391 -> 2392 ret = gca().legend(*args, **kwargs) 2393

Re: [matplotlib-devel] path simplification with nan (or move_to)

2008-10-08 Thread Michael Droettboom
John Hunter wrote: > On Wed, Oct 8, 2008 at 11:37 AM, Michael Droettboom <[EMAIL PROTECTED]> wrote: > > >> I figured this out. When this happens, a RuntimeError("Agg rendering >> complexity exceeded") is thrown. >> > > Do you think it is a good idea to put a little helper note in the > exc

Re: [matplotlib-devel] path simplification with nan (or move_to)

2008-10-08 Thread John Hunter
On Wed, Oct 8, 2008 at 11:37 AM, Michael Droettboom <[EMAIL PROTECTED]> wrote: > I figured this out. When this happens, a RuntimeError("Agg rendering > complexity exceeded") is thrown. Do you think it is a good idea to put a little helper note in the exception along the lines of throw "Agg re

Re: [matplotlib-devel] path simplification with nan (or move_to)

2008-10-08 Thread Michael Droettboom
Michael Droettboom wrote: > Eric Firing wrote: > >> Michael Droettboom wrote: >> >>> Eric Firing wrote: >>> Mike, John, Because path simplification does not work with anything but a continuous line, it is turned off if there are any nans in the path. The r

Re: [matplotlib-devel] path simplification with nan (or move_to)

2008-10-08 Thread Michael Droettboom
Eric Firing wrote: > Michael Droettboom wrote: >> Eric Firing wrote: >>> Mike, John, >>> >>> Because path simplification does not work with anything but a >>> continuous line, it is turned off if there are any nans in the >>> path. The result is that if one does this: >>> >>> import numpy as np

Re: [matplotlib-devel] path simplification with nan (or move_to)

2008-10-08 Thread Eric Firing
The patch in that last message of mine was clearly not quite right. I have gone through several iterations, and have seemed tantalizingly close, but I still don't have it right yet. I need to leave it alone for a while, but I do think it is important to get this working correctly ASAP--certai