matplotlib-devel Mailing List for matplotlib

I'm having some regression issues with 0.99.1.1 compared to 0.98.5.3 in 
"blocking_input.py".
I'm using pylab.clabel(...,manual=True) expecting my figure to ask me to 
input the locations of the contour labels. All was fine in 0.98.5.3, it 
does not seem to work with 0.99.1.1.
After some investigations I found:
1. With rev 7591, the button1, button2, and button3 methods in the 
BlockingMouseInput class have been renamed to mouse_event_add, 
mouse_event_stop, and mouse_event_stop. To date, these changes have not 
been made to BlockingContourLabeler which still has the button1 and 
button2 method that are in charge of actually adding and removing the 
new labels, and that are never called.
Replacing button1 by mouse_event_add and button3 by mouse_event_pop 
seems to return the "manually add contour label" functionality.
2. After that change, I get a crash on mouse_button_stop in 
BlockingMouseInput on line 196:
TypeError: stop_event_loop() takes exactly 1 argument (2 given)
After confirmation from the docuemntation that stoip_event_loop() should 
indeed not take any argument I would fix line 196 with:
 < self.fig.canvas.stop_event_loop(event)
 > self.fig.canvas.stop_event_loop()
-Julien

Andrew Straw wrote:
> David Cournapeau wrote:
> 
>> Hi,
>>
>> I don't know if that's of any interest for matplotlib developers,
>> but I added scripts to build matplotlib with numscons:
>>
>> http://github.com/cournape/matplotlib/tree/scons_build
>> 
>> 
> Hi David,
>
> I'm trying to download your git repository, but I'm having trouble
> cloning it.
>
> Can you do this? "git clone git://github.com/cournape/matplotlib.git"
> 
Yes, I have exactly the same problem - cloning through ssh is ok, though
(although it does not help you much). I already tried to remove/refork
the repo from you (I thought the problem may have been caused by
astraw/matplotlib being a new repo), to no end.
I wish git would improve their network support...
> For me, I get an error:
>
> $ git clone git://github.com/cournape/matplotlib.git
> Initialized empty Git repository in /home/astraw/tmp/mpl/matplotlib/.git/
> fatal: The remote end hung up unexpectedly
>
> This has been happening for 2 days now.
>
> If it's not working for you, can you file a support ticket with github?
> (Or just delete your old repo and start a new one on github, preferably
> forking from the astraw/matplotlib.git repo.)
> 
I will,
David

>> This is a bug report for matplotlib version 0.99.1.1
>>
>> The clip_path keyword of imshow() does not work when setting it to 
>> (Path, Transform) for two reasons:
> 
> Hi, Thanks for the report. Do you have a simple test script that we can
> use to see the problem and then fix it? We will then use this as the
> basis to create a test function to make sure the issue never "crops" up
> again.
> 
> -Andrew
Hi Andrew,
Please try the following. It's a script that clips an NxN image based on 
its contour.
Thanks,
-Julien
import numpy
import pylab
#Create a NxN image
N=100
(x,y) = numpy.indices((N,N))
x -= N/2
y -= N/2
r = numpy.sqrt(x**2+y**2-x*y)
#Create a contour plot at N/4 and extract both the clip path and transform
c=pylab.contour(r,[N/4])
clipPath = c.collections[0].get_paths()[0]
clipTransform = c.collections[0].get_transform()
#Plot the image clipped by the contour
pylab.imshow(r, clip_path=(clipPath,clipTransform))
pylab.show()

Gellule Xg wrote:
> This is a bug report for matplotlib version 0.99.1.1
> 
> The clip_path keyword of imshow() does not work when setting it to 
> (Path, Transform) for two reasons:
> 
> 1. On line 622 of artist.py, v is now the tuple (Path,Transform) and 
> should be apply to func with apply(func,v). Hence the following 
> suggested change:
> 
> for k,v in props.items():
> func = getattr(self, 'set_'+k, None)
> if func is None or not callable(func):
> raise AttributeError('Unknown property %s'%k)
> < func(v)
> > if type(v) is tuple:
> > apply(func,v)
> else:
> func(v)
apply is deprecated. The block above could be written in one line as
 func(*tuple(v))
Eric
> changed = True
> 
> 2. On line 6278 of axes.py, im.set_clip_path(self.patch) should not 
> overwrite the clip path if already set. Hence the following suggested 
> change:
> 
> im.set_data(X)
> im.set_alpha(alpha)
> self._set_artist_props(im)
> < im.set_clip_path(self.patch)
> > if not im._clipon:
> > im.set_clip_path(self.patch)
> #if norm is None and shape is None:
> # im.set_clim(vmin, vmax)
> if vmin is not None or vmax is not None:
> im.set_clim(vmin, vmax)
> else:
> im.autoscale_None()
> im.set_url(url)
> 
> ------------------------------------------------------------------------------
> Come build with us! The BlackBerry(R) Developer Conference in SF, CA
> is the only developer event you need to attend this year. Jumpstart your
> developing skills, take BlackBerry mobile applications to market and stay 
> ahead of the curve. Join us from November 9 - 12, 2009. Register now!
> http://p.sf.net/sfu/devconference
> _______________________________________________
> Matplotlib-devel mailing list
> Mat...@li...
> https://lists.sourceforge.net/lists/listinfo/matplotlib-devel

Gellule Xg wrote:
> This is a bug report for matplotlib version 0.99.1.1
> 
> The clip_path keyword of imshow() does not work when setting it to 
> (Path, Transform) for two reasons:
Hi, Thanks for the report. Do you have a simple test script that we can
use to see the problem and then fix it? We will then use this as the
basis to create a test function to make sure the issue never "crops" up
again.
-Andrew

This is a bug report for matplotlib version 0.99.1.1
The clip_path keyword of imshow() does not work when setting it to 
(Path, Transform) for two reasons:
1. On line 622 of artist.py, v is now the tuple (Path,Transform) and 
should be apply to func with apply(func,v). Hence the following 
suggested change:
 for k,v in props.items():
 func = getattr(self, 'set_'+k, None)
 if func is None or not callable(func):
 raise AttributeError('Unknown property %s'%k)
< func(v)
 > if type(v) is tuple:
 > apply(func,v)
 else:
 func(v)
 changed = True
2. On line 6278 of axes.py, im.set_clip_path(self.patch) should not 
overwrite the clip path if already set. Hence the following suggested 
change:
 im.set_data(X)
 im.set_alpha(alpha)
 self._set_artist_props(im)
< im.set_clip_path(self.patch)
 > if not im._clipon:
 > im.set_clip_path(self.patch)
 #if norm is None and shape is None:
 # im.set_clim(vmin, vmax)
 if vmin is not None or vmax is not None:
 im.set_clim(vmin, vmax)
 else:
 im.autoscale_None()
 im.set_url(url)