#12034: improve doctest coverage of matrix/benchmark.py from 0% to 100%
----------------------------------------------------+-----------------------
   Reporter:  was                                   |          Owner:  jason, 
was   
       Type:  enhancement                           |         Status:  
needs_info   
   Priority:  minor                                 |      Milestone:  sage-5.0 
    
  Component:  linear algebra                        |       Keywords:           
    
Work_issues:                                        |       Upstream:  N/A      
    
   Reviewer:  Martin Albrecht, Karl-Dieter Crisman  |         Author:  William 
Stein
     Merged:                                        |   Dependencies:           
    
----------------------------------------------------+-----------------------
Changes (by newvalueoldvalue):

  * status:  needs_review => needs_info
  * reviewer:  => Martin Albrecht, Karl-Dieter Crisman
  * author:  wstein => William Stein


Comment:

 Martin, does that mean you give positive review to William's patch?  It
 looks fine at a glance...  Anyway, putting "needs info" for this.

 Other than not obeying the recommendation from the developer guide about
 {{{.. note}}}
 {{{
 However, you are encouraged to use the upper-case version
 }}}
 which is not a problem at all, the reviewer patch looks fine, including
 the very minor code changes.  Just one or two very minor formatting
 things.

 I've attached a reviewer-reviewer patch which fixes these and adds this to
 the reference manual.

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