#8630: Cusp forms constructor ignores the character and returns enormous space
-----------------------------+----------------------------------------------
   Reporter:  davidloeffler  |       Owner:  craigcitro     
       Type:  defect         |      Status:  positive_review
   Priority:  blocker        |   Milestone:  sage-4.4       
  Component:  modular forms  |    Keywords:                 
     Author:                 |    Upstream:  N/A            
   Reviewer:                 |      Merged:                 
Work_issues:                 |  
-----------------------------+----------------------------------------------
Changes (by was):

  * status:  needs_review => positive_review


Comment:

 1. Below "ring" should be "modular symbols space":
 {{{
         117         def change_ring(self, R):
         118             r"""
         119             Return this ring with the base ring changed to the
 ring R.
 }}}

 2. Here I think the sentence should end with ::

 {{{
         419         A more complicated example involving both a nontrivial
 character, and a
         420         base field that is not minimal for that character:
 }}}


 It's really awesome that you sphinxified a bunch of docs, in addition to
 fixing the bug in this ticket.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8630#comment:6>
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.

Reply via email to