#8800: Doctest coverage of categories
--------------------------+-------------------------------------------------
Reporter: SimonKing | Owner: Simon King
Type: defect | Status: needs_review
Priority: major | Milestone: sage-4.4.2
Component: categories | Keywords: categories doctests
Author: Simon King | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
--------------------------+-------------------------------------------------
Comment(by robertwb):
Replying to [comment:28 SimonKing]:
> Hi Robert!
>
> Replying to [comment:27 robertwb]:
> > MatrixFunctor.__init__, is there not a module category that could be
used in place of {{{CommutativeAdditiveGroups}}}? I guess if tbe basering
is unknown then that's more difficult.
>
> Yes, {{{Modules()}}} requires a base ring. There is currently no
category of modules, but only a category of R-modules for any ring R. This
is why I used {{{CommutativeAdditiveGroups()}}} in several cases.
Hmm... does it make sense to have a category of Modules (over any
basering)?
> > Missing periods on {{{VectorFunctor.__cmp__}}} and
{{{VectorFunctor.merge}}}.
>
> Missing where? In the doc string?
Yes, there were a couple of sentences without ending periods. Nothing
major.
> Concerning positive review, note that technically this ticket depends on
#8807, which has no review yet.
>
Yep. I started to look at that one too, and will review it if no one beats
me too it when I have another spare moment (maybe the upcoming Sage days,
depending on how good of shape my thesis is in by then).
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8800#comment:29>
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.
