#18067: sage/graphs/graph.py: multigraph recognition in init fails
------------------------------------------+------------------------
Reporter: darij | Owner:
Type: defect | Status: new
Priority: major | Milestone: sage-6.6
Component: graph theory | Resolution:
Keywords: graphs, sage-combinat, | Merged in:
Authors: | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
Dependencies: | Stopgaps:
------------------------------------------+------------------------
Comment (by ncohen):
> Sorry, Nathann; I have no idea how to fix this mess short of replacing
the catch-all `__init__` by a bunch of separate constructors each of which
does one thing well and has a predictable return type.
Don't worry, I will write some code for that very soon. I am about to
leave to Liege (Belgium) for some Sage days so I will not be able to do it
this week-end most probably (I'll be backpacking) but it will definitely
be done during the week.
I would also like to turn this huge `__init__` into independent functions,
but that is not totally straightforward either. I do want to make it
shorter, though.
> Guessing the user's intent shouldn't happen unless the user asks for it.
I agree with you. But I already wrote something to make those argument
mandatory, and noticed that it broke doctests.. Because a complete graph
stored as a multigraph is not (for Sage) equal to a complete graph (stored
as a simple graph) and other problems like that.
Well. 1) I will fix this in the next days and 2) I am aware that this code
has to be changed, and I will do something about it. Especially since
David Coudert would like to be able to load huge graphs in Sage, and that
it is currently impossible.
Nathann
--
Ticket URL: <http://trac.sagemath.org/ticket/18067#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.
