#9395: new doctest for french book about Sage
---------------------------+------------------------------------------------
   Reporter:  zimmerma     |       Owner:  mvngu     
       Type:  enhancement  |      Status:  needs_info
   Priority:  major        |   Milestone:  sage-4.5.3
  Component:  doctest      |    Keywords:            
     Author:               |    Upstream:  N/A       
   Reviewer:               |      Merged:            
Work_issues:               |  
---------------------------+------------------------------------------------
Changes (by ylchapuy):

  * status:  needs_review => needs_info


Comment:

 Replying to [comment:6 zimmerma]:
 > Yann, please could you review this? What you have to do (William please
 correct me if needed):
 >
 > 1) [optional] check the content of the numbertheory.py file matches the
 corresponding chapter of the book. I write "optional" since this should be
 our (the authors of the book) responsability.
 >
 > 2) check the new doctests pass with the latest Sage version (we used
 4.4.4)
 >
 > Paul

 Sorry for the delay.
 All tests passed with Sage 4.5.2.

 Regarding the `timeit` issue, you could change
 {{{
     sage: %timeit (a^e) # not tested
 }}}
 for
 {{{
     sage: timeit('a^e') # random
 }}}

 (and also `# long` for some of them).

 I put it to `need info` for now, but feel free either to change this and
 ask me for a review (I'll be faster this time) or don't change anything
 and give a positive review.

  Yann

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