#26814 closed enhancement (fixed)
pep8 cleaning of clustering methods
Reported by:  David Coudert  Owned by:  

Priority:  major  Milestone:  sage8.6 
Component:  graph theory  Keywords:  
Cc:  Merged in:  
Authors:  David Coudert  Reviewers:  Frédéric Chapoton 
Report Upstream:  N/A  Work issues:  
Branch:  d83f778 (Commits, GitHub, GitLab)  Commit:  d83f778e9542ebd52ec97babe928a617b9687d3a 
Dependencies:  Stopgaps: 
Description
we do PEP8 cleaning of methods clustering_average
, clustering_coeff
and cluster_transitivity
.
Change History (11)
comment:1 Changed 4 years ago by
Branch:  → public/26814_clean_clustering_methods 

Commit:  → a97a9b7d4d8db8222ca0baa0e1e65338c6a72f7c 
Status:  new → needs_review 
comment:2 Changed 4 years ago by
do not do that
 TESTS:: + Empty graph::
because the TESTS section is hidden (not displayed)
comment:3 Changed 4 years ago by
Commit:  a97a9b7d4d8db8222ca0baa0e1e65338c6a72f7c → 11de534d25f13b98829009c044cd732f9f9bfa4d 

Branch pushed to git repo; I updated commit sha1. New commits:
11de534  trac #26814: reviewer's comment

comment:5 Changed 4 years ago by
Status:  needs_review → needs_work 

typo ValueEror
otherwise, looks good
comment:6 Changed 4 years ago by
Commit:  11de534d25f13b98829009c044cd732f9f9bfa4d → d83f778e9542ebd52ec97babe928a617b9687d3a 

comment:8 Changed 4 years ago by
Reviewers:  → Frédéric Chapoton 

Status:  needs_work → positive_review 
merci !
comment:10 Changed 4 years ago by
Branch:  public/26814_clean_clustering_methods → d83f778e9542ebd52ec97babe928a617b9687d3a 

Resolution:  → fixed 
Status:  positive_review → closed 
comment:11 Changed 4 years ago by
Milestone:  sage8.5 → sage8.6 

This tickets were closed as fixed after the Sage 8.5 release.
Note: See
TracTickets for help on using
tickets.
New commits:
trac #26814: pep8 cleaning in clustering_average, clustering_coeff and cluster_transitivity