fix issue in graph_plot_js

[dcoudert]

Fix issue reported in https://groups.google.com/forum/#!topic/sage-devel/oD1jQVirJBs

The mapping of vertices given to d3.js was incorrect

Component: graph theory

Author: David Coudert

Branch/Commit: 8e5f238

Reviewer: Frédéric Chapoton

Issue created by migration from https://trac.sagemath.org/ticket/27460

[dcoudert]

Commit: 771b91e

[dcoudert]

Branch: u/dcoudert/27460_graph_plot_js

[dcoudert]

New commits:

771b91e

trac #27460: fix ordering

[fchapoton]

comment:2

Having a doctest would be good..

[sagetrac-git]

Changed commit from 771b91e to ebddf9d

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

ebddf9d

trac #27460: add doctest

[dcoudert]

comment:4

I added a doctest but it's not good. It works on my laptop with py3, but not with py2.
The order of the keys of the dictionaries is not the same in py2 and py3 :(

with py2, I get

..."links": [{"strength": 0, "target": 0, "color": "#aaa", "curve": 0, "source": 1, "name": ""}], "pos": [], "edge_thickness": 4, "loops": [], "edge_labels": false, "vertex_size": 7, "vertex_labels": true, "directed": true, "gravity": 0.04, "charge": -120, "nodes": [{"group": "0", "name": "10"}, {"group": "0", "name": "1"}]...

and with py3

..."nodes": [{"name": "10", ...}, {"name": "1", ...}], "links": [{"source": 1, "target": 0, ...}]...

Any suggestion is more than welcome.

[sagetrac-git]

Changed commit from ebddf9d to 8e5f238

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

8e5f238

trac #27460: a more robust doctest

[dcoudert]

comment:6

OK, I found a more robust doctest that is OK for py2 and py3. Hope you like it.

[fchapoton]

comment:7

ok, looks good. When patchbot is green, positive review

[fchapoton]

Reviewer: Frédéric Chapoton

[vbraun]

Changed branch from u/dcoudert/27460_graph_plot_js to 8e5f238