[IPython-dev] Helping battle testing the newapp branch

MinRK benjaminrk at gmail.com
Mon Jun 20 23:38:21 EDT 2011

On Mon, Jun 20, 2011 at 20:09, Fernando Perez <fperez.net at gmail.com> wrote:
> On Mon, Jun 20, 2011 at 4:38 PM, MinRK <benjaminrk at gmail.com> wrote:
>> Is that a 'go' for merge? If so, I'll tag&merge once I get home in an
>> hour or so.  Or we can wait a day or so for objections.
> From my side, it's a go.  The ideas are sound, and we're at the point
> where what we need is battle testing to shake out the problems.  So I
> don't see holding on having much value at this point.

Okay, I will go ahead with the merge this evening, then.

Since this is merging newapp, I'll call the pre-merge tag 'oldapp'.

> Some minor comments:
> - This variable seems to be ignored now, it worked in master:
> c.Global.ignore_old_config = True

Nothing looks at the Global config anymore. This is now
application-level, in c.InteractiveShellApp.ignore_old_config I
believe (I don't see these warnings with my default config).  Perhaps
it should be moved to BaseIPythonApplication, though.

> for people who have a single ipython_dir (like me) because they need
> to handle both 0.10 and 0.11 series simultaneously, it's nice to have
> a way to silence all those warnings.
> - I don't think we should print the profile name in the default case,
> it's just noise.  I realize we now have a more consistent structure
> for profiles and even the default case is now a profile, but we should
> keep the amount of printed stuff to a minimum in the default cases.

Easy enough, I'll do this before merging.


> Otherwise, it looks good, passes the test suite, etc.
> Cheers,
> f

More information about the IPython-dev mailing list