Re: [matplotlib-devel] A small fix for backend specification...

2010-08-25 Thread Fernando Perez
On Wed, Aug 25, 2010 at 3:06 PM, Eric Firing wrote: > Looks fine to me.  It's fixing a bug.  I don't think the comment is even > necessary--the rationale looks pretty obvious, and the code is clear. > Great, thanks. I'll shorten the comment to just one line then: +# Lowercase only non-mo

Re: [matplotlib-devel] A small fix for backend specification...

2010-08-25 Thread Fernando Perez
On Wed, Aug 25, 2010 at 2:59 PM, Gael Varoquaux wrote: > > Freeking awesome! > > Go go team! Thanks :) We're pretty happy, we'll post more in a few weeks when there's something more solid to show. Take care, f -- Sell

Re: [matplotlib-devel] A small fix for backend specification...

2010-08-25 Thread Eric Firing
On 08/25/2010 11:57 AM, Fernando Perez wrote: > Hi folks, > > I'd like to know if the fix below looks reasonable to you, this is a > diff against current svn trunk: > > dreamweaver[matplotlib]> svn diff > Index: __init__.py > === > --

Re: [matplotlib-devel] A small fix for backend specification...

2010-08-25 Thread Gael Varoquaux
On Wed, Aug 25, 2010 at 02:57:50PM -0700, Fernando Perez wrote: > [1] teaser for the curious: > http://fperez.org/tmp/ipython_qt_pylab.png. All code is in the > 'newkernel' github branch. Special credits to Evan Patterson from > Enthought, the Qt brains behind the magic. Freeking awesome! Go go

[matplotlib-devel] A small fix for backend specification...

2010-08-25 Thread Fernando Perez
Hi folks, I'd like to know if the fix below looks reasonable to you, this is a diff against current svn trunk: dreamweaver[matplotlib]> svn diff Index: __init__.py === --- __init__.py (revision 8656) +++ __init__.py (working copy) @@