Opened 14 years ago
Last modified 8 years ago
#3572 needs_work enhancement
[with patch; needs work] optimize sage startup time by not importing any modules that import linbox by default.
Reported by: | was | Owned by: | cwitty |
---|---|---|---|
Priority: | major | Milestone: | sage-6.4 |
Component: | misc | Keywords: | |
Cc: | Merged in: | ||
Authors: | Reviewers: | ||
Report Upstream: | Work issues: | ||
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
Attachments (1)
Change History (10)
comment:1 Changed 14 years ago by
- Summary changed from optimize sage startup time by not importing any modules that import linbox by default. to [with patch; needs review] optimize sage startup time by not importing any modules that import linbox by default.
Changed 14 years ago by
comment:2 Changed 14 years ago by
comment:3 Changed 14 years ago by
- Summary changed from [with patch; needs review] optimize sage startup time by not importing any modules that import linbox by default. to [with patch; needs works] optimize sage startup time by not importing any modules that import linbox by default.
With this patch applied against 3.0.4.alpha2 I get one doctest failure:
mabshoff@sage:/scratch/mabshoff/release-cycle/sage-3.0.4.alpha2$ ./sage -t -long devel/sage/sage/misc/session.pyx sage -t -long devel/sage/sage/misc/session.pyx ********************************************************************** File "/scratch/mabshoff/release-cycle/sage-3.0.4.alpha2/tmp/session.py", line 212: sage: save_session('session', verbose=True) Expected: Not saving ... Got: Saving doctest Saving a Not saving example_2: example_2 is a function Not saving example_3: example_3 is a function Not saving example_0: example_0 is a function Not saving example_1: example_1 is a function Not saving example_4: example_4 is a function Not saving example_5: example_5 is a function Saving __file__ Not saving f: f is a function Saving __doc__ Saving __builtins__ Saving __name__ ********************************************************************** 1 items had failures: 1 of 10 in __main__.example_4 ***Test Failed*** 1 failures. For whitespace errors, see the file /scratch/mabshoff/release-cycle/sage-3.0.4.alpha2/tmp/.doctest_session.py [2.0 s] exit code: 1024
Cheers,
Michael
comment:4 Changed 14 years ago by
- Summary changed from [with patch; needs works] optimize sage startup time by not importing any modules that import linbox by default. to [with patch; needs work] optimize sage startup time by not importing any modules that import linbox by default.
comment:5 Changed 13 years ago by
The patch doesn't apply cleanly any more, as you might expect. I only got one doctest failure, from the following issue (not the one mabshoff reported, perhaps because that doctest in the patch didn't apply):
sage: os.system('sage -startuptime | grep linbox') sage.libs.linbox.linbox: 0.006 (sage.matrix.matrix_modn_dense) 0
So there's another import which needs to be removed...
comment:6 Changed 9 years ago by
- Milestone changed from sage-5.11 to sage-5.12
comment:7 Changed 9 years ago by
- Milestone changed from sage-6.1 to sage-6.2
comment:8 Changed 8 years ago by
- Milestone changed from sage-6.2 to sage-6.3
comment:9 Changed 8 years ago by
- Milestone changed from sage-6.3 to sage-6.4
Note: See
TracTickets for help on using
tickets.
on osx, BEFORE:
AFTER
Of course now,
but that makes a lot of sense. Only make people pay if they use the goods.
William