#6500: [with patch, needs review] Improve reference manual coverage for
polynomial
rings
---------------------------+------------------------------------------------
Reporter: davidloeffler | Owner: tba
Type: defect | Status: new
Priority: major | Milestone: sage-4.1.1
Component: documentation | Keywords:
Reviewer: | Author:
Merged: |
---------------------------+------------------------------------------------
Comment(by jhpalmieri):
Source code looks good and the documentation looks good. I'm attaching a
referee's patch which looks large, but the large majority of which
contains repetitions of two changes: "EXAMPLE::" --> "EXAMPLES::" and
".::" --> ". ::" (a period followed by :: produces ".:" in the Sphinx
documentation, while if you insert a space, no colons are printed). Of
the other changes, I think the only one worth mentioning is that I changed
instances of the "param" command used in polynomial_singular_interface.py
to INPUT blocks, for consistency with the rest of the documentation.
I don't think my patch needs much review, but I'll keep this marked as
"needs review". davidloeffler's two patches get positive reviews from me;
if my patch is okay, the whole thing gets a positive review.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6500#comment:5>
Sage <http://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
-~----------~----~----~----~------~----~------~--~---
