#6745: [with patch; nearly positive review] quaternion algebras -- add 
computation
of left and right orders associated to ideals
-------------------------+--------------------------------------------------
 Reporter:  was          |       Owner:  tbd       
     Type:  enhancement  |      Status:  new       
 Priority:  major        |   Milestone:  sage-4.1.2
Component:  algebra      |    Keywords:            
 Reviewer:               |      Author:            
   Merged:               |  
-------------------------+--------------------------------------------------

Comment(by robertwb):

 sage/algebras/quatalg/quaternion_algebra.py:1272

 {{{
 ALGORITHM: Let `b_1, b_2, b_3, b_3` be a basis for this
 }}}

 (Typo, you want b_4).

 TabularUnified  sage/matrix/matrix_integer_dense.pyx:2310

 {{{
 if max(self._nrows, self._ncols) <= 50:
 }}}

 I think you intended to have an {{{elif}}} here.

 Other than that, looks good to me. Also, while I was playing around with
 it trying it out, I found #6762 useful.

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