Opened 10 years ago
Last modified 9 months ago
#11043 positive_review defect
Lazily import plot.
Reported by: | robertwb | Owned by: | jason |
---|---|---|---|
Priority: | major | Milestone: | sage-duplicate/invalid/wontfix |
Component: | misc | Keywords: | sd32 |
Cc: | kcrisman | Merged in: | |
Authors: | Reviewers: | Dima Pasechnik | |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
Improve startup time.
Attachments (3)
Change History (16)
Changed 10 years ago by
comment:1 Changed 10 years ago by
- Description modified (diff)
- Status changed from new to needs_review
Changed 10 years ago by
comment:2 Changed 10 years ago by
- Status changed from needs_review to needs_work
comment:3 Changed 10 years ago by
- Status changed from needs_work to needs_review
This is a more conservative patch which only saves importing the plot.* modules (about a 50-100ms savings on my computer).
comment:4 Changed 10 years ago by
- Description modified (diff)
- Summary changed from Lazily import plot, numpy, etc. to Lazily import plot.
comment:5 Changed 10 years ago by
- Status changed from needs_review to needs_work
Robert, did you even test this? It seems to fail doctests all over the place. E.g.,
sage -t devel/sage/sage/calculus/riemann.pyx ********************************************************************** File "/mnt/usb1/scratch/wstein/sage-4.7.1-sage.math.washington.edu-x86_64-Linux/devel/sage-main/sage/calculus/riemann.pyx", line 114: sage: m = Riemann_Map([f, hf], [hf, hfprime], 0.5 + 0.5*I) Exception raised: Traceback (most recent call last): File "/mnt/usb1/scratch/wstein/sage-4.7.1-sage.math.washington.edu-x86_64-Linux/local/bin/ncadoctest.p ... ImportError: No module named riemann
It seems like using LazyImport? in any nontrivial way always results in serious breakage that is revealed by doctests. I hit this numerous times myself. Hmmm.
comment:6 Changed 10 years ago by
- Keywords sd32 added
comment:7 Changed 9 years ago by
Is anything happening on this? Robert wrote the patch, William says it breaks doctests all over the place, but then it just stalls.
Dave
comment:8 Changed 9 years ago by
- Cc kcrisman added
comment:9 Changed 8 years ago by
- Milestone changed from sage-5.11 to sage-5.12
comment:10 Changed 7 years ago by
- Milestone changed from sage-6.1 to sage-6.2
comment:11 Changed 7 years ago by
- Milestone changed from sage-6.2 to sage-6.3
comment:12 Changed 7 years ago by
- Milestone changed from sage-6.3 to sage-6.4
comment:13 Changed 9 months ago by
- Milestone changed from sage-6.4 to sage-duplicate/invalid/wontfix
- Reviewers set to Dima Pasechnik
- Status changed from needs_work to positive_review
outdated
Note: See
TracTickets for help on using
tickets.
Before
After
That's another quarter-second (12.5%) savings.