autopx magic broken #1024

Closed
jenshnielsen opened this Issue Nov 21, 2011 · 4 comments

Comments

Projects
None yet
3 participants
@jenshnielsen
Contributor

jenshnielsen commented Nov 21, 2011

I noticed that the parallel_mpi.ipynb example is broken in master.
More specifically autopx cannot be disabled and
returns get_ipython() cannot be found from individual engines.

I have bisected that the code broke with 7f451c7
where make_quoted_expr() was replaced by standard repr()

@takluyver

This comment has been minimized.

Show comment
Hide comment
@takluyver

takluyver Nov 21, 2011

Member

I'm puzzled as to the connection. @minrk, is there a way to debug errors occurring on the engines?

Member

takluyver commented Nov 21, 2011

I'm puzzled as to the connection. @minrk, is there a way to debug errors occurring on the engines?

@minrk

This comment has been minimized.

Show comment
Hide comment
@minrk

minrk Nov 21, 2011

Member

Yes, it's a one-character fix in a regex in the parallel magic, I'll push in a sec.

Member

minrk commented Nov 21, 2011

Yes, it's a one-character fix in a regex in the parallel magic, I'll push in a sec.

@minrk minrk closed this in b3785ba Nov 21, 2011

@minrk

This comment has been minimized.

Show comment
Hide comment
@minrk

minrk Nov 21, 2011

Member

fixed

Member

minrk commented Nov 21, 2011

fixed

@jenshnielsen

This comment has been minimized.

Show comment
Hide comment
@jenshnielsen

jenshnielsen Nov 21, 2011

Contributor

Yes fixed. Thanks

Contributor

jenshnielsen commented Nov 21, 2011

Yes fixed. Thanks

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

fix autopx regex after recent change
7f451c7 changed make_quoted_str to repr,
which allowed the string to be either single or double-quoted, which
the regex previously did not handle.

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