#12415: Update doctesting framework
-------------------------------------------------------------------------------------------------------------------------+
       Reporter:  robertwb                                                      
                                         |         Owner:  mvngu         
           Type:  enhancement                                                   
                                         |        Status:  needs_review  
       Priority:  major                                                         
                                         |     Milestone:  sage-5.1      
      Component:  doctest                                                       
                                         |    Resolution:                
       Keywords:                                                                
                                         |   Work issues:                
Report Upstream:  N/A                                                           
                                         |     Reviewers:  Jeroen Demeyer
        Authors:  David Roe, Robert Bradshaw                                    
                                         |     Merged in:                
   Dependencies:  #13147,#13146, #13145, #12723, #12392, #12393, #12395, 
#12396, #12397, #12381, #12382, #12383, #12384  |      Stopgaps:                
-------------------------------------------------------------------------------------------------------------------------+

Comment (by jdemeyer):

 The following is cut from a `make ptest` run:
 {{{
 sage -t devel/sage/sage/symbolic/expression.pyx
     [1960 tests, 38.2 s]
 sage -t
 
devel/sage/sage/pl**********************************************************************
 File "sage.schemes.elliptic_curves.ell_rational_field", line 1468, in
 sage.schemes.elliptic_curves.ell_rational_field.EllipticCurve_ration
 al_field.simon_two_descent
 Failed example:
     E.simon_two_descent()  # long time (9s on sage.math, 2011)
 Expected:
     (5, 5, [(5 : 8 : 1), (4 : 9 : 1), (3 : 11 : 1), (-1 : 20 : 1), (-6 :
 -25 : 1)])
 Got:
     Saturation index bound = 265
     WARNING: saturation at primes p > 97 will not be done;
     points may be unsaturated at primes between 97 and index bound
     Failed to saturate MW basis at primes [ ]
     Saturation index bound = 265
     WARNING: saturation at primes p > 199 will not be done;
     points may be unsaturated at primes between 199 and index bound
     Failed to saturate MW basis at primes [ ]
     (5, 5, [(5 : 8 : 1), (4 : 9 : 1), (3 : 11 : 1), (-1 : 20 : 1), (-6 :
 -25 : 1)])
 **********************************************************************
 1 items had failures:
    1 of  38 in
 
sage.schemes.elliptic_curves.ell_rational_field.EllipticCurve_rational_field.simon_two_descent
 ***Test Failed*** 1 failures.
 ot/plot3d/implicit_plot3d.py
     [57 tests, 26.9 s]
 sage -t devel/sage/sage/modular/modsym/ambient.py
     [397 tests, 26.4 s]
 }}}

 There are 3 issues:

 1. The doctest failure itself.

 2. The fact that it's tested when the test is marked `# long time` but the
 `--long` option is not given.

 3. Output flushing: note the
 {{{
 sage -t devel/sage/sage/pl
 }}}
 and then later
 {{{
 ot/plot3d/implicit_plot3d.py
     [57 tests, 26.9 s]
 }}}

 By all means please keep working on this, it already looks very cool!

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