#12036: improve doctest coverage of gsl/interpolation.pyx from 0% to 100%
-----------------------------------+----------------------------------------
   Reporter:  was                  |          Owner:  jason, jkantor
       Type:  enhancement          |         Status:  needs_info    
   Priority:  minor                |      Milestone:  sage-4.8      
  Component:  numerical            |       Keywords:                
Work_issues:                       |       Upstream:  N/A           
   Reviewer:  Karl-Dieter Crisman  |         Author:  William Stein 
     Merged:                       |   Dependencies:                
-----------------------------------+----------------------------------------
Changes (by newvalueoldvalue):

  * status:  needs_review => needs_info
  * reviewer:  => Karl-Dieter Crisman
  * author:  => William Stein


Comment:

 {{{
 sage -coverage sage/gsl/interpolation.pyx
 ----------------------------------------------------------------------
 sage/gsl/interpolation.pyx
 ERROR: Please add a `TestSuite(s).run()` doctest.
 SCORE sage/gsl/interpolation.pyx: 100% (9 of 9)
 ----------------------------------------------------------------------
 sage -t  "devel/sage-main/sage/gsl/interpolation.pyx"
          [17.8 s]

 ----------------------------------------------------------------------
 All tests passed!
 Total time for all tests: 18.0 seconds
 }}}
 I assume the error about the testsuite would be part of this other ticket.

 Also, the cdef methods are not tested (which I think is supposed to be
 okay), and neither is the {{{__dealloc__}}} method, which apparently is ok
 since it didn't show up on `sage -coverage`.


 ----

 Questions:
  * Should this be in the reference manual?
  * Should some of the examples in this patch (I really like the
 {{{__setitem__}}} examples) be in the main thing, so that `spline?` is
 more useful?

 If so, these might as well be taken care of now.

 Otherwise looks good.

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