too aggressive namespace cleaning when calling run from another run magic. (NameError) #238

Closed
ivanov opened this Issue Dec 29, 2010 · 7 comments

Comments

Projects
None yet
5 participants
Owner

ivanov commented Dec 29, 2010

as reported to the mailinglist here by unit

 #beginning of script.py
 import IPython
 ip = IPython.ipapi.get()
 for i in range(0,5):
           ip.magic('%run script_1.py')
 #end of the script.py
 In [1]: run script

will fail with a NameError after the first iteration (i==1, not i==0)

Owner

ivanov commented Dec 29, 2010

In [1]: run -i script

is a potential workaround

Owner

takluyver commented Mar 23, 2011

Confirmed in trunk

Owner

ivanov commented Apr 4, 2011

test for this bug is in trunk as of 94644f8.
You will find it in the file IPython/core/tests/test_run.py please remove the skip_known_failure decorator to work on this bug.

markvoorhies pushed a commit to markvoorhies/ipython that referenced this issue Apr 21, 2011

ivanov added a commit to ivanov/ipython that referenced this issue Jan 25, 2012

mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this issue Nov 3, 2014

@minrk minrk removed prio-low labels Jan 14, 2015

Owner

Carreau commented Jul 21, 2016

This has been open for 4 years, and the use case seem extremely restrictive. As I doubt anyone will work on that. I'm going to close this.

Feel free to open a new issue or reopen this one if you think this is still needed and/or plan to work on it.

Thanks.

PS: @ivanov we miss you.

@Carreau Carreau closed this Jul 21, 2016

Owner

ivanov commented Jul 22, 2016

Lol! Miss you guys too.

On Thu, Jul 21, 2016 at 12:01 PM, Matthias Bussonnier <
notifications@github.com> wrote:

This has been open for 4 years, and the use case seem extremely
restrictive. As I doubt anyone will work on that. I'm going to close this.

Feel free to open a new issue or reopen this one if you think this is
still needed and/or plan to work on it.

Thanks.

PS: @ivanov https://github.com/ivanov we miss you.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#238 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHNw8OtJf5BzuwDcaY6t0p3JsgoJ6nyks5qX8IPgaJpZM4JSHOV
.

Paul Ivanov
http://pirsquared.org | GPG/PGP key id: 0x0F3E28F7

Owner

willingc commented Jul 22, 2016

@ivanov @Carreau For your amusement, http://bugs.python.org/msg270877 We have at least 2 more years if we wish to keep it open. Keep open all the things! ;-)

Owner

ivanov commented Jul 22, 2016

Yeah, I feel the same about keeping all known bugs open - especially given that I think I added a test decorated as "known failure" to the codebase that tested for this at some point. But I didn't feel that strongly about it.

@minrk minrk modified the milestones: no action, wishlist May 2, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment