Thanks for the fix! I have a suggestion, though:
You changed "nodes" to go over the vertices in the order determined by the
hashmap.
I'd suggest to change "v_to_id" to go over the sorted vertices instead,
i.e. something on the line of:
V = G.vertices(sort=True)
v_to_id = {v: i for i, v in enumerate(V)}
nodes = [... for v in V]
That way you'd end up with the same result independent of the ordering in the
hashmap.
With that you can also construct more test cases that don't depend on that
ordering and work for python2 and python3.
Cheers,
Stefan
On Monday, 11 March 2019 09:07:36 UTC+1, david....@inria.fr wrote:
>
> Thank you for reporting this issue.
>
> The order of the vertices given to d3.js was incorrect. This is fixed in
> https://trac.sagemath.org/ticket/27460
>
--
You received this message because you are subscribed to the Google Groups
"sage-devel" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to sage-devel+unsubscr...@googlegroups.com.
To post to this group, send email to sage-devel@googlegroups.com.
Visit this group at https://groups.google.com/group/sage-devel.
For more options, visit https://groups.google.com/d/optout.
