Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[MRG+1] Fix format_stack with compiled code (e.g. .so or .pyd) in the traceback - #382

Merged
aabadie merged 1 commit into
joblib:masterfrom
lesteve:fix-format-stack-with-compiled-code
Aug 1, 2016
Merged

[MRG+1] Fix format_stack with compiled code (e.g. .so or .pyd) in the traceback#382
aabadie merged 1 commit into
joblib:masterfrom
lesteve:fix-format-stack-with-compiled-code

Conversation

@lesteve

@lesteve lesteve commented Jul 28, 2016

Copy link
Copy Markdown
Member

Fixes #101.

Took the fix from IPython.core.ultratb.py

@aabadie

aabadie commented Jul 28, 2016

Copy link
Copy Markdown
Contributor

@lesteve, in case you missed it, tests are broken on Travis under Python 3.3.

@lesteve
lesteve force-pushed the fix-format-stack-with-compiled-code branch 2 times, most recently from 9fe1751 to 2674adc Compare July 29, 2016 05:28
@aabadie

aabadie commented Jul 29, 2016

Copy link
Copy Markdown
Contributor

All green +1

@aabadie aabadie changed the title [MRG] Fix format_stack with compiled code (e.g. .so or .pyd) in the traceback [MRG+1] Fix format_stack with compiled code (e.g. .so or .pyd) in the traceback Jul 29, 2016
@lesteve

lesteve commented Jul 29, 2016

Copy link
Copy Markdown
Member Author

I forgot to add a comment about the addition of SyntaxError in the code, I'll do that soonish.

@aabadie

aabadie commented Jul 29, 2016

Copy link
Copy Markdown
Contributor

I forgot to add a comment about the addition of SyntaxError in the code, I'll do that soonish.

Ok, let me know when it's done.

@lesteve
lesteve force-pushed the fix-format-stack-with-compiled-code branch from 2674adc to 18f7fad Compare July 29, 2016 13:01
@lesteve
lesteve force-pushed the fix-format-stack-with-compiled-code branch from 18f7fad to 90deace Compare July 29, 2016 15:37
@lesteve

lesteve commented Aug 1, 2016

Copy link
Copy Markdown
Member Author

@aabadie I added the aforementioned comment and an entry in CHANGES.rst.

@aabadie

aabadie commented Aug 1, 2016

Copy link
Copy Markdown
Contributor

LGTM, merging. Thanks !

@aabadie
aabadie merged commit 169dd49 into joblib:master Aug 1, 2016
@lesteve
lesteve deleted the fix-format-stack-with-compiled-code branch August 1, 2016 08:16
@amueller

amueller commented Aug 2, 2016

Copy link
Copy Markdown
Contributor

Awesome, thanks!
I'd love to have this in sklearn 0.8.
@ogrisel thoughts?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

joblib doesn't propogate correct exception, raises SyntaxError instead

3 participants