#12495: Bring plot.py to 100% coverage
-----------------------------+----------------------------------------------
   Reporter:  kcrisman       |          Owner:  mvngu              
       Type:  enhancement    |         Status:  needs_review       
   Priority:  major          |      Milestone:  sage-5.0           
  Component:  documentation  |       Keywords:                     
Work_issues:                 |       Upstream:  N/A                
   Reviewer:  John Palmieri  |         Author:  Karl-Dieter Crisman
     Merged:                 |   Dependencies:                     
-----------------------------+----------------------------------------------
Changes (by jhpalmieri):

  * reviewer:  => John Palmieri


Comment:

 Looks mostly great, I just have a few small corrections; see the referee's
 patch. If you're happy with my changes, feel free to give the ticket a
 positive review.

 Not too important because underscore methods aren't included in the
 reference manual by default, but line 3863
 {{{
 We can make commands not display their plots by default.
 }}}
 should probably end with a double colon: `by default. ::`.  Same with line
 3888.

 Line 3899 (the last line reproduced here) is wrong:
 {{{
      def nrows(self):
          """
          String representation of the graphics array.
 }}}

 While fixing these, I also added a blank line after line 3114:
 {{{
     ``exclude`` and ``detect_poles`` can be used together::
         sage: f(x) = (floor(x)+0.5) / (1-(x-0.5)^2)
 }}}

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