#5171: [with patch, needs work] Overhaul the Graph and DiGraph initialization
functions.
--------------------------+-------------------------------------------------
Reporter: was | Owner: rlm
Type: defect | Status: assigned
Priority: critical | Milestone: sage-3.3
Component: graph theory | Keywords:
--------------------------+-------------------------------------------------
Comment(by was):
REPORT:
* All doctests pass in the graphs directory
* old doctests don't fail in too disturbing a way
* I definitely do not like at all the new multiple_edges function. It
should be split into two functions: has_multiple_edges() and
allows_multiple_edges() and the current multiple_edges should be
deprecated. I personally would expect a function named "multiple_edges"
to return a list of all multiple edges, which no variant of the current
function does! Having a check parameter that determines the functionality
is particularly bad, given that check is used elsewhere all over in sage
in a way that never changes the meaning of the output (it is always for
speed).
Otherwise I'm good with this patch.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5171#comment:8>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---
