#14138: some cleanup in sage.combinat.combinat
---------------------------------+------------------------------------------
Reporter: ncohen | Owner: sage-combinat
Type: defect | Status: closed
Priority: major | Milestone: sage-5.8
Component: combinatorics | Resolution: fixed
Keywords: | Work issues:
Report Upstream: N/A | Reviewers: Punarbasu Purkayastha,
Nicolas M. Thiéry
Authors: Nathann Cohen | Merged in: sage-5.8.beta1
Dependencies: | Stopgaps:
---------------------------------+------------------------------------------
Comment (by ncohen):
> Also for the record, #13605 was positively reviewed about the same time
as this one.
This ticket was set to `positive_review` 5 days ago, when Nicolas Thiery
changed it "needs_work" for 4 invalid reasons (you can read it, it is just
above) and a new doctest which says "the function does what it should". I
have yet to be surprised by the speed at which the combinat code is fixed,
so please do not blame me for not believing that your ticket would be
reviewed 5 days later, to inadvertently fix a problem which had been
laying around for 4 years.
I'm sorry for the trouble it created on your side. I just want to see
those problems fixed quickly. And twice, if needed.
Nathann
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14138#comment:30>
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?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.
