#11732: faster import of sage.interacts.all
-----------------------------+----------------------------------------------
Reporter: saraedum | Owner: itolkov, jason
Type: enhancement | Status: closed
Priority: minor | Milestone: sage-4.7.2
Component: interact | Resolution: fixed
Keywords: sd32 | Work_issues:
Upstream: N/A | Reviewer: William Stein
Author: Julian Rueth | Merged: sage-4.7.2.alpha3
Dependencies: |
-----------------------------+----------------------------------------------
Old description:
> ''sage.interact.all'' imports functions which have controls from
> ''sagenb.notebook.interact'' as default parameters. These take some time
> to initialize. The attached patch improves this situation:
>
> {{{
> $ dumbbench -- ./sage --startuptime
> Ran 23 iterations (3 outliers).
> Rounded run time per iteration: 1.001e+00 +/- 2.2e-02 (2.2%)
> }}}
>
> vs.
>
> {{{
> $ dumbbench -- ./sage --startuptime
> Ran 24 iterations (4 outliers).
> Rounded run time per iteration: 9.56e-01 +/- 1.7e-02 (1.8%)
> }}}
>
> ----
>
> Apply [attachment:trac_11732_sagenb_interact.patch] to the '''SageNB'''
> repository.
New description:
''sage.interact.all'' imports functions which have controls from
''sagenb.notebook.interact'' as default parameters. These take some time
to initialize. The attached patch improves this situation:
{{{
$ dumbbench -- ./sage --startuptime
Ran 23 iterations (3 outliers).
Rounded run time per iteration: 1.001e+00 +/- 2.2e-02 (2.2%)
}}}
vs.
{{{
$ dumbbench -- ./sage --startuptime
Ran 24 iterations (4 outliers).
Rounded run time per iteration: 9.56e-01 +/- 1.7e-02 (1.8%)
}}}
----
Apply only [attachment:trac_11732_sagenb_interact.proper.patch] to the
'''SageNB''' repository.
--
Comment(by leif):
I've attached a `*.proper.patch`, which is identical except that I removed
the "garbage" before "`# HG changeset patch`", i.e., I deleted the first
line "`exporting patch:`", since Jeroen's current merger rejects such
patches.
For now, please make sure all your patches start with "`# HG changeset
patch`", i.e., have it on the first line without any preceding messages or
whatever.
I've relaxed that in ''my version'' of the merger, but Jeroen and maybe
others are likely to use his more restrictive one.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11732#comment:8>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
