#1956: implement multivariate truncated power series arithmetic
-------------------------------------------+--------------------------------
   Reporter:  was                          |       Owner:  pernici              
    
       Type:  enhancement                  |      Status:  needs_review         
    
   Priority:  major                        |   Milestone:  sage-4.7             
    
  Component:  commutative algebra          |    Keywords:  multivariate power 
series
     Author:  Niles Johnson                |    Upstream:  N/A                  
    
   Reviewer:  Martin Albrecht, Simon King  |      Merged:                       
    
Work_issues:                               |  
-------------------------------------------+--------------------------------
Changes (by niles):

  * status:  needs_work => needs_review


Comment:

 Replying to [comment:101 hlaw]:
 >
 > Works great!  The quick turnaround is appreciated!
 >

 Glad to hear it!  If you feel capable of reviewing some part of the patch,
 that would be *greatly* appreciated.  Here is the current status of the
 various items:

 1. Sage passes all doctests with this patch
 (positive review; buildbot)

 2. All code is documented and doctested thoroughly; documentation builds
 without error or warning
 (needs review)

 3. The underlying concept of the implementation (a wrapper for certain
 univariate power series over multivariate polynomials) is sound
 (positive review; Mario Pernici)

 4. multi_power_series_ring.py: the code accurately does what it claims to
 do
 (needs review)

 5. multi_power_series_ring_element.py: the code accurately does what it
 claims to do
 (needs review)

 6. Integration with the rest of sage: construction and use of
 `PowerSeriesRings` works correctly, and parallels behavior of polynomial
 rings
 (needs review)

 7. Performance: the multivariate power series arithmetic is fast enough to
 be included in Sage
 (positive review; Mario Pernici)

 8. Coding: the code is free from obvious inefficiencies in error handling,
 memory management, etc.
 (needs review)

 9. The items on this list constitute a complete review
 (needs review)

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