Do not import IPython at startup

Question

Do not import IPython at startup

jdemeyer opened this issue 10 years ago · 8 comments

Just a few changes to do this.

This patch also removes the broken functions log_html, log_dvi, log_text from the global namespace (no point in deprecating them if they don't work)

Component: performance

Author: Jeroen Demeyer

Branch/Commit: 3f2192a

Reviewer: John Palmieri

Issue created by migration from https://trac.sagemath.org/ticket/18726

Answer 1 · 2015-06-18T08:29:57.000Z

Description changed:

--- 
+++ 
@@ -1 +1 @@
-This function causes IPython to be imported at startup, but it's not even used!
+Just a few changes to do this.

Answer 2 · 2015-06-18T09:11:38.000Z

Branch: u/jdemeyer/remove_load_ipython_extension__

Answer 3 · 2015-06-18T10:03:22.000Z

New commits:

`3f2192a`	`Do not load IPython at startup`

Answer 4 · 2015-06-18T10:03:22.000Z

Commit: 3f2192a

Answer 5 · 2015-06-18T10:04:45.000Z

Description changed:

--- 
+++ 
@@ -1 +1,3 @@
 Just a few changes to do this.
+
+This patch also removes the broken functions `log_html, log_dvi, log_text` from the global namespace (no point in deprecating them if they don't work)

Answer 6 · 2015-06-18T18:52:02.000Z

Reviewer: John Palmieri

Answer 7 · 2015-06-18T18:52:02.000Z

comment:5

This looks good, IPython is not loaded at startup, and all tests pass.

Answer 8 · 2015-06-20T09:20:17.000Z

Changed branch from u/jdemeyer/remove_load_ipython_extension__ to 3f2192a