Opened 14 years ago
Closed 14 years ago
#5059 closed defect (fixed)
[with patch, positive review] Fix a bunch of broken pickles
Reported by: | craigcitro | Owned by: | craigcitro |
---|---|---|---|
Priority: | major | Milestone: | sage-3.3 |
Component: | modular forms | Keywords: | |
Cc: | Merged in: | ||
Authors: | Reviewers: | ||
Report Upstream: | Work issues: | ||
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
At a workshop in Seattle last June, a massive number of spaces of modular symbols were computed. However, the pickles were broken in shortly thereafter by a refactoring of some code in Sage.
The attached patch fixes this -- now the old pickles can be loaded, and new pickles still work fine.
Attachments (2)
Change History (7)
Changed 14 years ago by
comment:1 Changed 14 years ago by
comment:2 Changed 14 years ago by
- Summary changed from [with patch, needs review] Fix a bunch of broken pickles to [with patch, positive review] Fix a bunch of broken pickles
Some examples: http://sage.math.washington.edu/home/wstein/db/modsym/data/
Looks good and works great.
comment:3 Changed 14 years ago by
- Summary changed from [with patch, positive review] Fix a bunch of broken pickles to [with patch, needs work] Fix a bunch of broken pickles
Unfortunately this patch breaks two doctests:
sage -t -long devel/sage/sage/modular/congroup.py # 5 doctests failed sage -t -long devel/sage/sage/modular/congroup_element.py # 1 doctests failed
Cheers,
Michael
Changed 14 years ago by
comment:4 Changed 14 years ago by
- Summary changed from [with patch, needs work] Fix a bunch of broken pickles to [with patch, positive review] Fix a bunch of broken pickles
That seems to have addressed those doctest failures.
comment:5 Changed 14 years ago by
- Resolution set to fixed
- Status changed from new to closed
Merged in Sage 3.3.alpha0
Cheers,
Michael
Note: See
TracTickets for help on using
tickets.
Are there some example broken pickles that this fixes?