#3749: [with patch; needs work] Request for a method "is_cyclic" for groups in
SAGE
--------------------------+-------------------------------------------------
 Reporter:  ljpk          |        Owner:  joyner    
     Type:  defect        |       Status:  new       
 Priority:  minor         |    Milestone:  sage-3.2.2
Component:  group_theory  |   Resolution:            
 Keywords:                |  
--------------------------+-------------------------------------------------
Comment (by wdj):

 In reply to

 {{{
 Is this actually necessary:

         665             if 1 in edivs:
         666                 edivs.remove(1)
 }}}

 As elementary_divisors is implemented:

 {{{
 sage: J = AbelianGroup([2,3])
 sage: J.invariants()
 [2, 3]
 sage: J.elementary_divisors()
 [1, 6]
 }}}

 But we probably should have
 {{{
 sage: J = AbelianGroup([2,3])
 sage: J.invariants()
 [2, 3]
 sage: J.elementary_divisors()
 [6]
 }}}

 since you want the elementary divisor of AbelianGroup([2,3]) to be the
 same as that of AbelianGroup([6]).

 I'll try to fix this too.

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