Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

warning message in IPython.extensions test #1062

Closed
jstenar opened this Issue · 11 comments

3 participants

@jstenar
Collaborator

I get a warning message printed during the IPython.extensions test.

C:\python\external> python .\ipython\IPython\scripts\iptest IPython.extensions
....ERROR: Magic function `store` not found....
----------------------------------------------------------------------
Ran 7 tests in 6.557s

OK

On python2.6, windows 7, ipython master 65ac74e

@takluyver
Owner

I see this as well. I think it's a dodgy doctest.

@fperez
Owner

I've actually been thinking, shouldn't we perhaps just turn %store back on up by default? It's a very useful extension that many people have asked for... Just a thought.

@takluyver
Owner

Yes, it had occurred to me. Do we have a mechanism for having extensions always enabled? Is it sufficient just to add it to the default extensions list?

I realise it doesn't have to be in an extension, but I quite like the extension system as a way to keep related functionality together. In particular, we can add functionality without explicitly telling the core how to initialise it.

We'd also need to make the autorestore behaviour configurable, and probably off by default.

@fperez
Owner

If you can whip out a PR on this, I'd be happy to review it.. I'm going to put out a beta and will have to bow out of ipython for a couple of days (I've fallen badly behind on work stuff), but this would be easy to review between beta and release time. Even if we don't have configuration yet, at least we'd be back to 0.10-level functionality (the loss of %store was strictly speaking a regression).

@takluyver
Owner

See PR #1073.

@jstenar
Collaborator

The warning is gone for me now.

@takluyver
Owner

@jstenar: With my storemagic-plugin branch, or in master?

@jstenar
Collaborator
@takluyver
Owner

OK, great, thanks.

@jstenar
Collaborator

I do not see this warning on a current master

@jstenar jstenar closed this
@fperez
Owner

Great, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.