#4762: [with patch, needs review] Odd error message for congruence subgroups
---------------------------+------------------------------------------------
 Reporter:  ljpk           |        Owner:  craigcitro
     Type:  defect         |       Status:  new       
 Priority:  minor          |    Milestone:  sage-3.2.2
Component:  modular forms  |   Resolution:            
 Keywords:                 |  
---------------------------+------------------------------------------------
Changes (by AlexGhitza):

  * summary:  Odd error message for congruence subgroups => [with patch,
              needs review] Odd error message for congruence
              subgroups

Comment:

 This is due to two underlying problems:

  * coercing an element of a congruence subgroup into the set of 2x2
 integer matrices is broken
  * if g is an element of a congruence subgroup, testing g == 0 is broken

 The attached patch fixes both issues.  Note that if !CongruenceSubgroup
 were to inherit from !MatrixGroup (which it probably should?), then the
 first problem would just go away.

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

Reply via email to