Opened 2 years ago
Closed 2 years ago
#27202 closed enhancement (fixed)
py3: fix doctests in random graphs generators
Reported by:  dcoudert  Owned by:  

Priority:  major  Milestone:  sage8.7 
Component:  graph theory  Keywords:  py3, graph 
Cc:  Merged in:  
Authors:  David Coudert  Reviewers:  Frédéric Chapoton 
Report Upstream:  N/A  Work issues:  
Branch:  2da5cec (Commits, GitHub, GitLab)  Commit:  2da5cec5ff882c17921cf890f863f088eb0bce17 
Dependencies:  Stopgaps: 
Description
We change doctests in RandomBarabasiAlbert
, RandomNewmanWattsStrogatz
, RandomHolmeKim
, RandomTreePowerlaw
, RandomRegular
and RandomShell
.
Several failing doctests in py3 were marked as 32bit
or 64bit
. Instead of adding extra tags, we change the doctests to more robust once, i.e., not depending of the precise list of edges.
We also take the opportunity to clean the methods.
Change History (7)
comment:1 Changed 2 years ago by
 Branch set to public/27202_random_graphs_generators
 Commit set to c5a8294883c9eab340352c5334aa30dd01a1130a
 Status changed from new to needs_review
comment:2 Changed 2 years ago by
 Commit changed from c5a8294883c9eab340352c5334aa30dd01a1130a to b33884695e419039200f4fe46229a2c1b7750e8b
Branch pushed to git repo; I updated commit sha1. New commits:
b338846  trac #27202: fix doctest in RandomBarabasiAlbert

comment:3 Changed 2 years ago by
at least one patchbot was not happy with
+ sage: G.degree_sequence() + [4, 3, 3, 2, 2, 2]
Maybe this should be removed or tagged #random ?
comment:4 Changed 2 years ago by
 Commit changed from b33884695e419039200f4fe46229a2c1b7750e8b to 2da5cec5ff882c17921cf890f863f088eb0bce17
Branch pushed to git repo; I updated commit sha1. New commits:
2da5cec  trac #27202: tag a doctest # random

comment:5 Changed 2 years ago by
I tagged it # random
.
comment:6 Changed 2 years ago by
 Reviewers set to Frédéric Chapoton
 Status changed from needs_review to positive_review
ok
comment:7 Changed 2 years ago by
 Branch changed from public/27202_random_graphs_generators to 2da5cec5ff882c17921cf890f863f088eb0bce17
 Resolution set to fixed
 Status changed from positive_review to closed
New commits:
trac #27202: fix doctests in generators/random.py