#16738: Shortening some very long cluster doctests
-------------------------+-------------------------------------------------
Reporter: | Owner:
chapoton | Status: needs_review
Type: | Milestone: sage-6.3
enhancement | Resolution:
Priority: major | Merged in:
Component: | Reviewers: Christian Stump
documentation | Work issues:
Keywords: | Commit:
cluster | fe6fea5058e414bcdb9fae3be686e4222c9367aa
Authors: | Stopgaps:
Frédéric Chapoton |
Report Upstream: N/A |
Branch: |
u/chapoton/16738 |
Dependencies: |
-------------------------+-------------------------------------------------
Comment (by chapoton):
Hello Christian,
thanks for your time. I think it is ready for review. I have kept only the
simplest tests about rank 2, marking the longer ones with {{{# not
tested}}}. I think these functions were used to check thoroughly that
there was no mistake in the code. One should keep them in the code to be
able to run them if some mistake appears at some point, but it does not
look necessary to run them every time.
I have also taken this opportunity to do some little code formatting, like
* lines of length at most 80
* pep8 standard for code (no space after or before parenthesis, etc)
* removed a lot of unused import statements (found with pyflakes)
* comment out 2 unused variables
--
Ticket URL: <http://trac.sagemath.org/ticket/16738#comment:4>
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 unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
