Opened 20 months ago
Closed 20 months ago
#26711 closed enhancement (fixed)
avoid .vertices() in graph_coloring.py
Reported by:  dcoudert  Owned by:  

Priority:  major  Milestone:  sage8.5 
Component:  graph theory  Keywords:  py3, graph 
Cc:  tscrim  Merged in:  
Authors:  David Coudert  Reviewers:  Frédéric Chapoton 
Report Upstream:  N/A  Work issues:  
Branch:  80f42b3 (Commits)  Commit:  80f42b3e44d0752c509a43482b0da99aea43934a 
Dependencies:  Stopgaps: 
Description
After #26484, one call to .vertices()
was remaining in method edge_coloring
. As it is used for the relabeling of a clique, we can safely replace it with list(h)
.
In this ticket, we also do another round of PEP8 cleaning and improvements of math formula for the html doc.
Change History (7)
comment:1 Changed 20 months ago by
 Branch set to public/26711_graph_coloring
 Cc tscrim added
 Commit set to 3e1f0dee2e30efc099764ae9db1a23eaa65f98ec
 Keywords py3 graph added
 Status changed from new to needs_review
comment:2 Changed 20 months ago by
 Branch changed from public/26711_graph_coloring to public/26711_graph_coloring_2
 Commit changed from 3e1f0dee2e30efc099764ae9db1a23eaa65f98ec to 50374e7b98d94610c3025c80aa4b87b78471b33a
I though I was in a new branch, but my changes were in develop
... I need another coffee... Should be OK now.
New commits:
50374e7  trac #26711: more cleaning in graph_coloring.py

Version 0, edited 20 months ago
by
(next)
comment:3 Changed 20 months ago by
 Reviewers set to Frédéric Chapoton
+ sage: sorted(d.keys())
could be just
+ sage: sorted(d)
Otherwise, positive review.
comment:4 Changed 20 months ago by
 Commit changed from 50374e7b98d94610c3025c80aa4b87b78471b33a to 80f42b3e44d0752c509a43482b0da99aea43934a
comment:5 Changed 20 months ago by
I have implemented you comment.
comment:7 Changed 20 months ago by
 Branch changed from public/26711_graph_coloring_2 to 80f42b3e44d0752c509a43482b0da99aea43934a
 Resolution set to fixed
 Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
New commits:
trac #26711: further cleaning in graph_coloring.py