Hello everybody !!
I had the bad idea to test Sage's graph library with a -optional
keyword, to notice *many* doctests failed because of the recent update
to NetworkX. I first believed that all default edges labels had been
replaced from None to {}, to finally notice only some of them were.
Right now, in sage :
sage: graphs.PetersenGraph().edges()
[(0, 1, None), (0, 4, None), (0, 5, None), (1, 2, None), (1, 6, None),
(2, 3, None), (2, 7, None), (3, 4, None), (3, 8, None), (4, 9, None),
(5, 7, None), (5, 8, None), (6, 8, None), (6, 9, None), (7, 9, None)]
sage: graphs.CompleteBipartiteGraph(3,4).edges()
[(0, 3, {}), (0, 4, {}), (0, 5, {}), (0, 6, {}), (1, 3, {}), (1, 4,
{}), (1, 5, {}), (1, 6, {}), (2, 3, {}), (2, 4, {}), (2, 5, {}), (2,
6, {})]
For now my patch to fix the doctests ( #8892 ) tests whether the
labels are different from None or {}, but this is not really a
solution..... What do you think of it ? Do we change the default label
to None, like before ? Do we change it to {} ? What would you advise ?
As there are many, many failures coming from this, it is not that easy
to test any other patch properly ^^;
Nathann
--
To post to this group, send an email to sage-devel@googlegroups.com
To unsubscribe from this group, send an email to
sage-devel+unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/group/sage-devel
URL: http://www.sagemath.org
