matplotlib-devel Mailing List for matplotlib

On Friday 27 January 2006 09:50, John Hunter wrote:
> >>>>> "Darren" == Darren Dale <dd...@co...> writes:
>
>     Darren> I would like to use verbose.report during the rc
>     Darren> validation process to fix this. The problem is that
>     Darren> verbose is initialized as 'silent', and is only changed to
>     Darren> the user-selected level after the rest of the rc-file has
>     Darren> been read. One solution might be to move the verbose
>     Darren> entries to the top of matplotlibrc, another might be to
>     Darren> modify __init__.py to validate the verbose settings first,
>     Darren> no matter where they are located in the matplotlibrc. I
>     Darren> favor the second approach.
>
>     Darren> Thoughts, opinions, suggestions?
>
> I am not in favor of moving them to the top, since I like to have the
> most commonly changed ones there.  Nor am I wild about a 2 pass
> approach.
>
> Would it work for you to buffer the rc messages during initialization,
> and then output them at the end of the rc read depending on the
> verbose settings?

changed in cvs

Fernando,
This looks like you're using the Tk backend.  This is one of the problems 
with trying to use a very common method name in widgets like 
resize().  Different GUI packages can define it differently.  It looks like 
Tk uses this for it's resize event handling while Gtk and Qt use resize( w, 
h ) for controlling size and other method names for the event handling like 
resizeEvent().

After some digging, I think this looks like it might be a bug (or at least 
a head ache) in the Tk backend design.  In backend_bases.py, there is this 
code:

class FigureCanvasBase:
    ...
     def resize(self, w, h):
         """
         set the canvas size in pixels
         """
         pass

Presumably, this is sort of a default implementation for a virtual method 
that others can rely on.  However, in backends/backend_tkagg.py we have this:

class FigureCanvasTkAgg(FigureCanvasAgg):
    ...
     def resize(self, event):
         width, height = event.width, event.height
     ...

And if you look in backends/backend_agg.py you'll see this:

class FigureCanvasAgg(FigureCanvasBase):
    ...

So FigureCanvasTkAgg is inheriting from FigureCanvasAgg which inherits from 
FigureCanvasBase which has a resize( w, h ) method.  However, 
FigureCanvasTkAgg re-implements resize with a different signature.  At best 
this very confusing.

It looks like FigureCanvasTkAgg.resize is used as a callback in the ctor 
for that class for one of the Tk events.  Here's what I'd suggest:

1) Rename FigureCanvasTkAgg.resize( event ) to FigureCanvasTkAgg.resize( w, 
h ).
2) Move the code that extracts the w, h from the event to a new method 
FigureCanvasTkAgg.resizeEvent( event ) like this:

class FigureCanvasTkAgg:
     def resizeEvent(self, event):
         width, height = event.width, event.height
         if self._resize_callback is not None:
             self._resize_callback(event)
         self.resize( width, height )

3) Change the FigureCanvasTkAgg ctor to use the resizeEvent callback 
instead of resize.

WARNING: I don't know anything about Tk!  I'm hoping someone that does now 
Tk can check this over and make sure it sounds right.

This of course doesn't actually do anything for fixing the problem we've 
been discussing about how to resize a plot after it's been created and have 
the window update accordingly.

Ted

At 11:08 AM 1/25/2006, Fernando Perez wrote:
>Ted Drain wrote:
>>Maybe one of you guys could refresh my memory.  What is the calling 
>>sequence we're going for?
>
>My original message was this:
>
>============================================================================
>In [1]: gcf().set_figsize_inches((8,8),forward=True)
>---------------------------------------------------------------------------
>exceptions.TypeError                                 Traceback (most recent
>call last)
>
>/home/fperez/code/python/pylab/arrows/<ipython console>
>
>/usr/lib/python2.3/site-packages/matplotlib/figure.py in
>set_figsize_inches(self, *args, **kwargs)
>      266             canvasw = w*dpival
>      267             canvash = h*dpival
>--> 268             self.canvas.resize(int(canvasw), int(canvash))
>      269
>      270     def get_size_inches(self):
>
>TypeError: resize() takes exactly 2 arguments (3 given)
>
>
>A quick look at the backends code shows this:
>
>      def resize(self, event):
>          width, height = event.width, event.height
>          self.toolbar.configure(width=width) # , height=height)
>
>So quite obviously, this doesn't work: it's expecting an event object, and a
>pair of numbers is being passed.
>
>I'm not sure what the proper fix should be here, I don't really know the code
>flow well enough.
>
>I should also note that the gcf().set_figsize_inches((8,8),forward=True) seems
>to produce a different on-screen result per backend (in some it doesn't do
>anything, in Qt it stretches the figure only horizontally, ...)  That code
>seems to be pretty much broken.
>
>I noticed that figure(figsize=(8,8)) seems to work fine, but I'm not sure how
>to programmatically resize an existing figure, given the above problems.
>============================================================================
>
>Beyond this, I'll leave it to the backend experts as to what the right 
>choices should be.  I just noted that shipping a feature _known_ to break 
>on all but one backend doesn't sound like the best approach :)
>
>Cheers,
>
>f

Ted Drain     Jet Propulsion Laboratory   ted...@jp...    

On Friday 27 January 2006 09:50, John Hunter wrote:
> >>>>> "Darren" == Darren Dale <dd...@co...> writes:
>
>     Darren> I would like to use verbose.report during the rc
>     Darren> validation process to fix this. The problem is that
>     Darren> verbose is initialized as 'silent', and is only changed to
>     Darren> the user-selected level after the rest of the rc-file has
>     Darren> been read. One solution might be to move the verbose
>     Darren> entries to the top of matplotlibrc, another might be to
>     Darren> modify __init__.py to validate the verbose settings first,
>     Darren> no matter where they are located in the matplotlibrc. I
>     Darren> favor the second approach.
>
>     Darren> Thoughts, opinions, suggestions?
>
> I am not in favor of moving them to the top, since I like to have the
> most commonly changed ones there.  Nor am I wild about a 2 pass
> approach.
>
> Would it work for you to buffer the rc messages during initialization,
> and then output them at the end of the rc read depending on the
> verbose settings?

That's a good idea. I'll work on it as soon as I get a chance, probably 
tomorrow morning.

Darren

>>>>> "Darren" == Darren Dale <dd...@co...> writes:

    Darren> I would like to use verbose.report during the rc
    Darren> validation process to fix this. The problem is that
    Darren> verbose is initialized as 'silent', and is only changed to
    Darren> the user-selected level after the rest of the rc-file has
    Darren> been read. One solution might be to move the verbose
    Darren> entries to the top of matplotlibrc, another might be to
    Darren> modify __init__.py to validate the verbose settings first,
    Darren> no matter where they are located in the matplotlibrc. I
    Darren> favor the second approach.

    Darren> Thoughts, opinions, suggestions?

I am not in favor of moving them to the top, since I like to have the
most commonly changed ones there.  Nor am I wild about a 2 pass
approach.

Would it work for you to buffer the rc messages during initialization,
and then output them at the end of the rc read depending on the
verbose settings?

JDH

On Friday 27 January 2006 07:04, Darren Dale wrote:
> On Friday 27 January 2006 6:58 am, Nils Wagner wrote:
> > Is it possible to disable this message when importing matplotlib ?
> >
> > >>> import matplotlib
> >
> > /usr/lib64/python2.4/site-packages/matplotlib/__init__.py:729:
> > UserWarning: ghostscript-8.15 found. ghostscript-8.16 or later is
> > recommended for use with the text.usetex option.
> >   warnings.warn( 'ghostscript-%s found. ghostscript-%s or later is \
>
> Not at present. I wanted to make this warning a call to verbose.report, but
> couldnt get it to work. The report was not printed. I'll look into it again
> this morning.

I would like to use verbose.report during the rc validation process to fix 
this. The problem is that verbose is initialized as 'silent', and is only 
changed to the user-selected level after the rest of the rc-file has been 
read. One solution might be to move the verbose entries to the top of 
matplotlibrc, another might be to modify __init__.py to validate the verbose 
settings first, no matter where they are located in the matplotlibrc. I favor 
the second approach.

Thoughts, opinions, suggestions?