<br><br><div class="gmail_quote">On Wed, Aug 15, 2012 at 12:13 PM, Thomas Kluyver <span dir="ltr">&lt;<a href="mailto:takowl@gmail.com" target="_blank">takowl@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hi Matt,<br>
<div class="im"><br>
On 15 August 2012 18:15, Matt Anderson &lt;<a href="mailto:manderso@broadcom.com">manderso@broadcom.com</a>&gt; wrote:<br>
&gt; To remedy, should the implementation of this function call InteractiveShellEmbed.instance() instead of InteractiveShellEmbed()?<br>
<br>
</div>Looking at the code for embed(), it has these three lines:<br>
<br>
    global _embedded_shell<br>
    if _embedded_shell is None:<br>
        _embedded_shell = InteractiveShellEmbed(**kwargs)<br>
<br>
So it&#39;s effectively implementing a singleton pattern itself. It looks<br>
to me like it would make more sense to use the .instance() API that&#39;s<br>
used elsewhere in the code, but I&#39;m not familiar with the reason it<br>
was originally done a different way.<br></blockquote><div><br></div><div>embed was implemented before the singleton stuff came together.  It probably should use instance(), but doing so has one disadvantage: It would break the ability to call embed() from within an existing IPython session.  I don&#39;t know if anyone cares about that use case, though.</div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I think Fernando might be most familiar with that bit of the code.<br>
He&#39;s very busy at the moment, so it might be a few days before he can<br>
chime in.<br>
<br>
Thanks,<br>
Thomas<br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
IPython-dev mailing list<br>
<a href="mailto:IPython-dev@scipy.org">IPython-dev@scipy.org</a><br>
<a href="http://mail.scipy.org/mailman/listinfo/ipython-dev" target="_blank">http://mail.scipy.org/mailman/listinfo/ipython-dev</a><br>
</div></div></blockquote></div><br>