#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 ncohen):
Hellooooooo !
> What do you mean by the comment that those functions look similar? They
compute similar, but not identical, information from the matroid. And all
four are useful to end users.
Nonono, of course you need them. I was just saying that perhaps there
could have been an internal function computing all 4 things, exposed in 4
differents ways to the user. This way there is no copy/paste of code.
> I guess we should have gotten rid of commented out parts (though Sage
has plenty of that floating around in its source). I'll revise that soon.
Nonono I'm sorry I said that, I will give you lengthier reviews soon.
Otherwise it will make you update the patch every day... `:-)`
Nathann
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7477#comment:18>
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.
