#19385: Refactor DiGraph.__init__
-------------------------+-------------------------------------------------
Reporter: | Owner:
ncohen | Status: needs_review
Type: | Milestone: sage-6.9
enhancement | Resolution:
Priority: major | Merged in:
Component: graph | Reviewers: Travis Scrimshaw
theory | Work issues:
Keywords: | Commit:
Authors: | 99c8601b947b746cfd0cbbca4c836b82b3ac5dde
Nathann Cohen | Stopgaps:
Report Upstream: N/A |
Branch: |
u/ncohen/19385 |
Dependencies: |
#19381 |
-------------------------+-------------------------------------------------
Comment (by ncohen):
Of course, if the matrix has weights different from +1-1 BUT the graph is
considered to have multiple edges, then those weights will be considered
as multiple edges.
This constructor is full of corner cases like that. Where we try to guess
what exactly the user did not know he wanted.
Half of the code sets flags depending on the values of others. And I am
sick of trying to analyse corner-cases where you are really not sure of
what should happen.
Nathann
--
Ticket URL: <http://trac.sagemath.org/ticket/19385#comment:16>
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.
