#12264: eisenstein series mod p -- speed regression due to "temporary
workaround"
---------------------------+------------------------------------------------
Reporter: was | Owner: tbd
Type: enhancement | Status: new
Priority: minor | Milestone: sage-5.0
Component: performance | Keywords:
Work_issues: | Upstream: N/A
Reviewer: | Author:
Merged: | Dependencies:
---------------------------+------------------------------------------------
Three years ago Alex Ghitza introduced a patch in #5102 (that wstein
positively reviewed) that said:
{{{
# this is a temporary fix due to a change in the
# polynomial constructor over finite fields; this
# is a notable speed regression, to be fixed soon.
return a0fac*R(eisenstein_series_poly(k, prec).list(), prec=prec,
check=True)
}}}
in the file {{{modular/modform/eis_series.py}}}. At the time, no ticket
was opened, and now it's over three years later and this has not "been
fixed soon". It should be!
I noticed this when working on the closely related #11375.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12264>
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.
