Opened 13 years ago
Closed 13 years ago
#8107 closed enhancement (fixed)
Fewer unnecessary imports from `sage.server.*`
Reported by: | mpatel | Owned by: | was |
---|---|---|---|
Priority: | minor | Milestone: | sage-4.3.2 |
Component: | notebook | Keywords: | |
Cc: | robertwb, timdumol | Merged in: | sage-4.3.2.alpha1 |
Authors: | Mitesh Patel | Reviewers: | Robert Bradshaw |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
This should reduce startup time, according to sage -startuptime
.
Attachments (1)
Change History (7)
Changed 13 years ago by
comment:1 Changed 13 years ago by
- Cc timdumol added
- Status changed from new to needs_review
The sage-repository patch and #8102 together reduce the overall time for me from ~1.9 s to ~1.7 s.
comment:2 Changed 13 years ago by
I'm sure we can do better with #7502...
comment:3 Changed 13 years ago by
- Status changed from needs_review to positive_review
Awesome -- I was just being annoyed by precisely these imports last night.
comment:4 follow-up: ↓ 5 Changed 13 years ago by
A big +1 from me too. I'm curious about the comment
# We import the following two only for doctesting purposes
though.
comment:5 in reply to: ↑ 4 Changed 13 years ago by
comment:6 Changed 13 years ago by
- Merged in set to sage-4.3.2.alpha1
- Resolution set to fixed
- Reviewers set to Robert Bradshaw
- Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
sage repo.