#4062: [with patch, needs review] Problems with Eisenstein series code?
---------------------------+------------------------------------------------
Reporter: craigcitro | Owner: craigcitro
Type: defect | Status: assigned
Priority: major | Milestone: sage-3.2
Component: modular forms | Resolution:
Keywords: |
---------------------------+------------------------------------------------
Comment (by was):
Further remarks from craig:
{{{
I posted a fix for the Eisenstein series bug that was reported a
little bit ago. (Kevin, I'm cc'ing you because William said he
mentioned this bug to you, too.) The original poster was right: his
snippet of code takes a character chi, asks for a weight two
Eisenstein series f in M_k(Gamma_1(N), chi), and then asks for
f.character() -- and Sage says that it isn't chi! So that was the bug.
The fix he starts detailing in his original post is completely the
wrong direction -- he's somehow trying to correct the series to match
the character that's getting returned. He later realized that the
right fix was actually to change the character returned.
}}}
and from Kevin:
{{{
The only comment I had about the report was that it sounded to me that the
poster had perhaps misunderstood the port of the theorem in Miyake to Sage
---he seemed to be saying "this character should be replaced by its
conjugate in several places", not realising that the notation in the sage
code was that the character in the code sounded to me like it was by
definition the conjugate of the character in Miyake.
}}}
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4062#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
-~----------~----~----~----~------~----~------~--~---