#8327: Implement the universal cyclotomic field, using Zumbroich basis
-----------------------------------------------------+----------------------
Reporter: nthiery | Owner:
Type: enhancement | Status:
needs_review
Priority: major | Milestone: sage-5.6
Component: number fields | Resolution:
Keywords: Cyclotomic field, Zumbroich basis | Work issues:
Report Upstream: N/A | Reviewers:
Authors: Christian Stump, Simon King | Merged in:
Dependencies: #13727, #13728 | Stopgaps:
-----------------------------------------------------+----------------------
Comment (by nbruin):
It looks like you have a memory leak in `cached_lcm`: that cache never
gets cleared out. Can you argue that it will only be called with a limited
variety of arguments so that you know the cache won't grow indefinitely?
Also, you should cache by set, not by tuple, since lcm isn't dependent on
order or multiplicity. That can already save you a factor ''n''! of
memory.
(but really, unless you can argue the argument set here is going to be
very limited, I don't think you can afford to build this cache)
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8327#comment:150>
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.
