clean generic_graph.py (part 14) - visualization

[David Coudert]

We clean (PEP8 and avoid .vertices / .edges) methods related to graph visualization: _color_by_label, latex_options, set_latex_options, layout, layout_spring, layout_ranked, layout_extend_randomly, layout_circular, layout_tree, layout_graphviz, _layout_bounding_box, _circle_embedding, _line_embedding, graphplot, _rich_repr_, plot, show, plot3d, show3d, _keys_for_vertices, graphviz_string, graphviz_to_file_named, and tachyon_vertex_plot.

Remaining calls to .vertices:

• layout_circular calls _circle_embedding with a list of vertices ordered by .vertices. Changing this will change the layout.

• _keys_for_vertices returns a mapping vertex to unique string identifier as required by dot2tex. It uses the order of self.vertices() to build the mapping. It is then used in graphviz_string and layout_graphviz.

Any advice for these 2 cases is more than welcome ;)

[David Coudert]

New commits:

​5d971aa

trac #26680: part 14 - visualization

[Frédéric Chapoton]

one failing doctest, and one plugin warning (missing r''')

[git]

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

​04c09d8	trac #26680: Merged with 8.5.beta6
​5f72b6e	trac #26680: fix plugin warning

[David Coudert]

I fixed the plugin warning, but I don't find the failing doctest. Which one is it ?

[Frédéric Chapoton]

in the patchbot:

sage -t --long --warn-long 55.5 src/sage/graphs/generic_graph.py
**********************************************************************
File "src/sage/graphs/generic_graph.py", line 19671, in sage.graphs.generic_graph.GenericGraph.?
Failed example:
    P.plot3d(engine='tachyon', edge_colors=edge_colors).show() # long time
Exception raised:
    Traceback (most recent call last):
      File "/home/chapoton/sage/local/lib/python2.7/site-packages/sage/doctest/forker.py", line 671, in _run
        self.compile_and_execute(example, compiler, test.globs)
      File "/home/chapoton/sage/local/lib/python2.7/site-packages/sage/doctest/forker.py", line 1086, in compile_and_execute
        exec(compiled, globs)
      File "<doctest sage.graphs.generic_graph.GenericGraph.?[20]>", line 1, in <module>
        P.plot3d(engine='tachyon', edge_colors=edge_colors).show() # long time
      File "/home/chapoton/sage/local/lib/python2.7/site-packages/sage/graphs/generic_graph.py", line 19788, in plot3d
        for u,v,l in edge_colors[color]:
    TypeError: 'int' object is not iterable

[git]

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

​e2cd65a

trac #26680: fix doctest in plot3d

[David Coudert]

ok, fixed.

[Frédéric Chapoton]

ok, let it be. But..

• not a good idea (rather a very bad idea) to mix many trivial pep8 changes with non trivial code changes. A few pep8 changes are ok. I myself try hard to refrain to do that, as tempting as it would be. Let us hope that your graph tickets will be reviewed nevertheless.

• pep8 should not be enforced in the doctests, no way. It's only for the code.

[David Coudert]

May be I should split the other tickets into easier to review tickets... Some of them are way too big.