#12949: Better congruence testing for odd arithmetic subgroups
----------------------------------+----------------------------------
       Reporter:  davidloeffler   |         Owner:  craigcitro
           Type:  enhancement     |        Status:  needs_review
       Priority:  major           |     Milestone:  sage-5.11
      Component:  modular forms   |    Resolution:
       Keywords:                  |     Merged in:
        Authors:  David Loeffler  |     Reviewers:  Vincent Delecroix
Report Upstream:  N/A             |   Work issues:
         Branch:                  |  Dependencies:
       Stopgaps:                  |
----------------------------------+----------------------------------
Changes (by davidloeffler):

 * status:  needs_work => needs_review
 * work_issues:  test =>


Comment:

 Here's a new patch. The link now points to a copy Kurth's homepage as
 mirrored on an archive website. I kept the randomized test as it was, but
 flagged it with "long"; the problem is that if you reduce the index too
 far, then you only ever get one of a very small set of subgroups and so
 it's not really a worthwhile test.

 I also corrected a tiny parity issue in the test routine (there are no odd
 subgroups of index 2 mod 4, and it was pure luck that the default
 parameters for the test routine don't provoke this error).

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