#16319: Implement k-closures for matroids
-------------------------------------+-------------------------------------
       Reporter:  tscrim             |        Owner:  sage-combinat
           Type:  enhancement        |       Status:  positive_review
       Priority:  minor              |    Milestone:  sage-6.3
      Component:  matroid theory     |   Resolution:
       Keywords:  line closure, k    |    Merged in:
  closure                            |    Reviewers:  Stefan van Zwam
        Authors:  Travis Scrimshaw   |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:                     |  1f49c59b9180cbc0b49d4b6a3cf7c628a74a6098
  public/matroids/k_closure-16319    |     Stopgaps:
   Dependencies:                     |
-------------------------------------+-------------------------------------
Changes (by Stefan):

 * status:  needs_review => positive_review
 * reviewer:   => Stefan van Zwam


Comment:

 The code looks fine. I'm worried about the speed of the Subsets method you
 use, but what you have implemented seems to work, so someone can write
 faster code as needed. It's good to go in!

--
Ticket URL: <http://trac.sagemath.org/ticket/16319#comment:3>
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.

Reply via email to