#5648: [with patch, needs review] Multiplication for modular forms
---------------------------+------------------------------------------------
Reporter: davidloeffler | Owner: davidloeffler
Type: enhancement | Status: assigned
Priority: minor | Milestone: sage-3.4.2
Component: modular forms | Keywords:
---------------------------+------------------------------------------------
Comment(by was):
Quick comment: I'm concerned the Sturm bound isn't enough when they are
Eisenstein series (it's definitely enough for cusp forms):
{{{
889 m = newparent.sturm_bound()
890 newqexp = self.qexp(m) * other.qexp(m)
891
892 return
newparent.base_extend(newqexp.base_ring())(newqexp)
}}}
Maybe I'm just being dense at the moment.
You could remedy this by increasing m if a given value doesn't work.
There might (should) be a method on newparent that returns the actual
precision needed to determine a q-expansion.
I am happy with all the API changes you list in the ticket summary,
including changing the meaning of equals for Dirichlet characters.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5648#comment:2>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---
