#7716: sage -coverage enhancement
-----------------------------+----------------------------------------------
Reporter: roed | Owner: mvngu
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-4.3
Component: documentation | Keywords: coverage
Work_issues: | Author: roed
Upstream: N/A | Reviewer:
Merged: |
-----------------------------+----------------------------------------------
Comment(by jhpalmieri):
Overall looks good. A few comments: It doesn't look like this will detect
Sphinx/reST markup for input and output, as described
[http://sagemath.org/doc/developer/conventions.html#documentation-strings
here] -- a block like
{{{
:param x: the length of the rectangle
:type x: float
:param w: the width of the rectangle
:type w: float
:return: the area of the rectange
:rtype: float
}}}
Or am I missing something?
Also, as I've said on #4323, it takes a certain amount of hubris, or maybe
(as mabshoff pointed out) just a strong sense of irony, to put functions
with no docstrings into a file like "sage-coverage".
Finally, I couldn't get it to apply cleanly, either. When applying to the
scripts repository in 4.3.rc0, I got the message
{{{
applying /Users/palmieri/Downloads/7716_coverage.patch
patching file sage-coverage
Hunk #2 FAILED at 15
1 out of 3 hunks FAILED -- saving rejects to file sage-coverage.rej
}}}
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7716#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.
