#12092: get doctest coverage of databases/stein_watkins.py to 100%
-----------------------------------+----------------------------------------
   Reporter:  minz                 |          Owner:  mvngu          
       Type:  enhancement          |         Status:  needs_work     
   Priority:  minor                |      Milestone:  sage-5.0       
  Component:  doctest              |       Keywords:                 
Work_issues:                       |       Upstream:  N/A            
   Reviewer:  Karl-Dieter Crisman  |         Author:  Moritz Minzlaff
     Merged:                       |   Dependencies:                 
-----------------------------------+----------------------------------------
Changes (by kcrisman):

 * cc: was (added)
  * reviewer:  => Karl-Dieter Crisman
  * status:  needs_review => needs_work


Comment:

 In general, this looks good.  However, I have a suspicion that it may not
 actually work.  Note that all the original doctests at the top have
 {{{
 # optional - stein_watkins_database
 }}}
 Should that be added to the others?  Likely.

 I agree about {{{_lines}}}.  It's not used anywhere.
 {{{

 sage: search_src('\._lines')
 databases/stein_watkins.py:202:        sage: l =
 sage.databases.stein_watkins._lines("\n some \ntext")
 geometry/cone.py:392:        cone._lines = lines
 geometry/cone.py:2765:            self._lines = lines
 geometry/cone.py:2766:        return self._lines
 geometry/polyhedra.py:3054:            return self._lines
 geometry/polyhedra.py:3056:            self._lines = [list(x) for x in
 self.line_generator()]
 geometry/polyhedra.py:3057:            return self._lines
 interfaces/qepcad.py:2183:        self._lines = lines
 }}}
 This has been in Sage practically from the beginning, though.  cc:ing
 William, but I suppose we can leave it in for now.
 {{{
 changeset:   0:039f6310c6fe
 user:        [email protected]
 date:        Sat Feb 11 01:13:08 2006 +0000
 summary:     [project @ original sage-0.10.12]
 }}}

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