#5460: [with patch, needs work] deprecate 'invert' in matrix_rational_dense
----------------------------+-----------------------------------------------
 Reporter:  jhpalmieri      |       Owner:  jhpalmieri
     Type:  defect          |      Status:  new       
 Priority:  minor           |   Milestone:  sage-3.4.1
Component:  linear algebra  |    Keywords:            
----------------------------+-----------------------------------------------

Comment(by jason):

 Good work!

 Do the doctests pass for the invert function?  If so, I'm surprised; I
 thought that the doctest output would have to include the deprecation
 statement.

 Regardless, the documentation for the invert function should include a
 sentence about the deprecation too.

 Positive review (based on rbeezer's positive review) pending the changes
 to documentation and possibly doctests mentioned above.

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