-
Notifications
You must be signed in to change notification settings - Fork 76
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use IPython #400
Use IPython #400
Conversation
It could be interesting to benchmark the starting time with and without this change, see if loading IPython affects start time. |
This is just a simple test for now. Only code execution works with the display hook. |
This could allow getting rid of a lot of code in xeus-python: error traceback, inspection, completion, etc. It would also bring magics support for free, even though some work would still be required to support matplotlib. |
7c497ac
to
02cc410
Compare
61530b5
to
f150900
Compare
What remains to have a mergeable PR:
|
2411974
to
3848528
Compare
4003dea
to
b7e7c98
Compare
29b91de
to
a43a7f0
Compare
a43a7f0
to
7a1d47f
Compare
Wow, this is a really big step for usability. Awesome work! |
Epic! |
Fix #62, fix #63, partially fix #224, fix #28, fix #359, fix #399, fix #402