#7477: Matroids
----------------------------------------------------+-----------------------
Reporter: ncohen | Owner: jkantor
Type: enhancement | Status:
needs_review
Priority: major | Milestone: sage-5.10
Component: combinatorics | Resolution:
Keywords: | Work issues:
Report Upstream: N/A | Reviewers:
Authors: Stefan van Zwam, Rudi Pendavingh | Merged in:
Dependencies: | Stopgaps:
----------------------------------------------------+-----------------------
Comment (by kcrisman):
Here are two alternate approaches to such a patch bomb.
* Ask the authors to parcel it out into manageable chunks. Given that
they have their own bitbucket repository, hopefully it wouldn't be
horribly difficult to separate out e.g. the examples, the core
functionality, the extended functionality, and then make this a
metaticket.
* Have the authors partially review it for each other; if there are truly
3-5 separate developers, they may be able to vouch for each others' code.
I think in this case some non-matroid-project person such as Nathann
should look at the overall structure, but I agree that doing it all
together is nearly impossible.
I, too, was very impressed with the general layout of this project; a lot
of thought has gone into this.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7477#comment:22>
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.
