Code review comment for lp:~ipython-dev/ipython/set_trace

Revision history for this message
Ville M. Vainio (villemvainio) wrote :

On Tue, Jul 28, 2009 at 3:12 AM, Fernando Perez<email address hidden> wrote:

>> import IPython; IPython.embed()
>
> Yes, I'm also OK with having a simplified embed() be part of the
> top-level public API. But it should at least take a few arguments.

IPython.ipapi is the top level public api, not IPython. Starting to
shove stuff to IPython package namespace is messy, and the line should
be drawn somewhere.

If the aim is to reduce typing, provide a root level module that gives
you access to stuff in IPython package (import ipy; ipy.embed() )

--
Ville M. Vainio
http://tinyurl.com/vainio

« Back to merge proposal