#15265: Additional functionality for Farey symbols
-------------------------------------+-------------------------------------
       Reporter:  hmonien            |        Owner:  hmonien
           Type:  enhancement        |       Status:  needs_work
       Priority:  major              |    Milestone:  sage-6.1
      Component:  modular forms      |   Resolution:
       Keywords:  Farey symbol,      |    Merged in:
  arithmetic subgroups               |    Reviewers:  David Loeffler,
        Authors:  Hartmut Monien     |  Martin Raum
Report Upstream:  N/A                |  Work issues:  LaTeX display doesn't
         Branch:                     |  work
  u/mraum/ticket/15265               |       Commit:
   Dependencies:                     |  7d78f7c0811682fbae3e9adb4b6e796241bd8b67
                                     |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by mraum):

 * reviewer:  David Loeffler => David Loeffler, Martin Raum


Comment:

 Hartmut, could you review the few changes that I made to your code? I have
 the following questions/comments, which you could help me with:

 reference for rademacher_matrix in farey.cpp?
 farey.cpp:FareySymbol(istream& is) why not "is >> *this" ?
 farey.cpp:370 : it seems that x will always have length 1.
 farey.cpp:129 I understand this is tempting, but PRETTY_FUNCTION is gcc
 only.
 farey_symbol.pyx:plot rgbcolor option does not apply to Dedekind
 tesselation.  Add one example each

 I still need to test the code, but hopefully I'll have time for this
 tomorrow.

--
Ticket URL: <http://trac.sagemath.org/ticket/15265#comment:17>
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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to