#4681: [with patches, partial positive review] General Smith normal form
implementation
---------------------------------------+------------------------------------
 Reporter:  davidloeffler              |        Owner:  davidloeffler
     Type:  enhancement                |       Status:  assigned     
 Priority:  major                      |    Milestone:  sage-3.2.2   
Component:  linear algebra             |   Resolution:               
 Keywords:  matrix, smith normal form  |  
---------------------------------------+------------------------------------
Changes (by davidloeffler):

  * summary:  [with patch, positive review] General Smith normal form
              implementation => [with patches, partial
              positive review] General Smith normal form
              implementation

Comment:

 Actually, after your comments on sage-devel, regarding the lack of
 doctests over base rings other than number field orders, I have done a
 third patch adding these. I meant to upload this yesterday but trac was
 down at the time. It also now raises an error over non-exact base rings,
 because I tested it with a 2x2 random matrix over CC and it got stuck in
 an infinite loop; and it always chooses 1 as a generator of the unit ideal
 so the results look nicer over fields. See the above third patch, to be
 applied on top of the previous two.

 Apologies for the profusion of patches!

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