Opened 2 years ago

Closed 2 years ago

#26821 closed enhancement (fixed)

pep8 cleaning in distance_graph and girth

Reported by: dcoudert Owned by:
Priority: major Milestone: sage-8.7
Component: graph theory Keywords:
Cc: Merged in:
Authors: David Coudert Reviewers: Bryan Gin-ge Chen
Report Upstream: N/A Work issues:
Branch: 5f9565a (Commits, GitHub, GitLab) Commit: 5f9565a75874db037237a4d28188cad75579d94b
Dependencies: Stopgaps:

Status badges

Description

pep8 cleaning in methods distance_graph and girth. Also, use iterators over edges or neighbors when appropriate.

Change History (5)

comment:1 Changed 2 years ago by dcoudert

  • Branch set to public/26821_distance_graph_and_girth
  • Commit set to 947a8fb7dc15d097a42bb3bee679059fa2580fc8
  • Status changed from new to needs_review

New commits:

​947a8fbtrac #26821: pep8 in distance_graph and girth

comment:2 Changed 2 years ago by git

  • Commit changed from 947a8fb7dc15d097a42bb3bee679059fa2580fc8 to 5f9565a75874db037237a4d28188cad75579d94b

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

​5f9565atrac #26821: Merged with 8.7.beta0

comment:3 Changed 2 years ago by dcoudert

  • Milestone changed from sage-8.5 to sage-8.7

comment:4 Changed 2 years ago by gh-bryangingechen

  • Reviewers set to Bryan Gin-ge Chen
  • Status changed from needs_review to positive_review

This looks good to me!

comment:5 Changed 2 years ago by vbraun

  • Branch changed from public/26821_distance_graph_and_girth to 5f9565a75874db037237a4d28188cad75579d94b
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.