#5272: [with patch, positive review] extend sage_input to work with RIF, CIF, 
AA,
and QQbar
--------------------+-------------------------------------------------------
 Reporter:  cwitty  |        Owner:  cwitty  
     Type:  defect  |       Status:  new     
 Priority:  major   |    Milestone:  sage-3.3
Component:  misc    |   Resolution:          
 Keywords:          |  
--------------------+-------------------------------------------------------
Changes (by mhampton):

  * summary:  [with patch, needs review] extend sage_input to work with
              RIF, CIF, AA, and QQbar => [with patch,
              positive review] extend sage_input to work with
              RIF, CIF, AA, and QQbar

Comment:

 This appears to work as intended, and passes doctests for me.

 In examining the coverage of these files, the additions are well
 documented and tested, but the remaining coverage of some of them are
 quite low.  I will file some tickets about that once I check that they
 don't already exist.

 I am giving this a positive review but it wouldn't hurt if someone who
 uses this code looked at it - I have not used sage_input before.  I don't
 think that's a reason not to put it in a release candidate though.

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