Skip to content
This repository has been archived by the owner on Aug 4, 2020. It is now read-only.

Pycassa Shell should change the way it starts IPython #193

Closed
donspaulding opened this issue Apr 8, 2013 · 1 comment
Closed

Pycassa Shell should change the way it starts IPython #193

donspaulding opened this issue Apr 8, 2013 · 1 comment

Comments

@donspaulding
Copy link

Django recently committed a fix from an IPython core dev that changed the way they started up IPython. It seems IPython does not encourage the use of embed() for full-blown shells, because it can create problems with locals() and globals(), and it appears Pycassa has a bug like this.

Pycassa may want to consider implementing this new method of instantiating an IPython shell.

try:
    from IPython.frontend.terminal.ipapp import TerminalIPythonApp
    app = TerminalIPythonApp.instance()
    app.initialize(argv=[])
    app.start()
except ImportError:
    # fall back to embed()?
@thobbs
Copy link
Member

thobbs commented Apr 10, 2014

I gave this a shot, but couldn't get it to work. If anybody else would like to try, please feel free to open a pull request.

@thobbs thobbs closed this as completed Apr 10, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants