Hi folks,

On Sat, Apr 17, 2010 at 3:42 PM, Minh Nguyen <nguyenmi...@gmail.com> wrote:

<SNIP>

> This issue is now tracked at ticket #8699:
>
> http://trac.sagemath.org/sage_trac/ticket/8699

This ticket now awaits review. The two patches on that ticket provide
documentation for the doctest coverage script, in addition to allowing
that script to handle docstrings that are delimited by triple single
quotes. Incidentally, using those patches I found that throughout the
whole Sage library, only one method uses triple single quotes. The
method in question is BubbleSortGraph() of the module
sage.graphs.graph_generators.py. This means that for the 90% doctest
coverage goal of Sage 5.0, we have one less method to document because
it's already documented. See the following command line transcript:

[mv...@sage sage-4.4.alpha0-8699-quotes]$ ./sage -coverageall >
coverage-before.log
[mv...@sage sage-4.4.alpha0-8699-quotes]$ cd local/bin/
[mv...@sage bin]$ hg qimport
http://trac.sagemath.org/sage_trac/raw-attachment/ticket/8699/trac_8699-documentation.patch
&& hg qpush
adding trac_8699-documentation.patch to series file
applying trac_8699-documentation.patch
now at: trac_8699-documentation.patch
[mv...@sage bin]$ hg qimport
http://trac.sagemath.org/sage_trac/raw-attachment/ticket/8699/trac_8699-single-quotes.patch
&& hg qpush
adding trac_8699-single-quotes.patch to series file
applying trac_8699-single-quotes.patch
now at: trac_8699-single-quotes.patch
[mv...@sage bin]$ cd ../..
[mv...@sage sage-4.4.alpha0-8699-quotes]$ ./sage -coverageall >
coverage-after.log
[mv...@sage sage-4.4.alpha0-8699-quotes]$ diff -Naur
coverage-before.log coverage-after.log
--- coverage-before.log 2010-04-17 03:37:50.663727239 -0700
+++ coverage-after.log  2010-04-17 03:38:45.181442502 -0700
@@ -381,7 +381,7 @@
 geometry/polytope.py: 27% (6 of 22)
 geometry/polyhedra.py: 100% (186 of 186)
 graphs/graph_bundle.py: 100% (5 of 5)
-graphs/graph_generators.py: 98% (73 of 74)
+graphs/graph_generators.py: 100% (74 of 74)
 graphs/planarity.pyx: 100% (1 of 1)
 graphs/graph_latex.py: 100% (10 of 10)
 graphs/schnyder.py: 100% (8 of 8)
@@ -1185,6 +1185,6 @@

 Overall weighted coverage score:  81.6%
 Total number of functions:  25377
-We need  852 more function to get to 85% coverage.
-We need 2121 more function to get to 90% coverage.
-We need 3390 more function to get to 95% coverage.
+We need  851 more function to get to 85% coverage.
+We need 2120 more function to get to 90% coverage.
+We need 3388 more function to get to 95% coverage.

-- 
Regards
Minh Van Nguyen

-- 
To post to this group, send an email to sage-devel@googlegroups.com
To unsubscribe from this group, send an email to 
sage-devel+unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/group/sage-devel
URL: http://www.sagemath.org

Reply via email to