Opened 3 years ago

Last modified 3 years ago

#25489 needs_work defect

cleaning graphs.RandomRegular

Reported by: vdelecroix Owned by:
Priority: major Milestone: sage-8.3
Component: documentation Keywords: MathExp2018
Cc: tmonteil Merged in:
Authors: Vincent Delecroix Reviewers:
Report Upstream: N/A Work issues:
Branch: u/vdelecroix/25489 (Commits, GitHub, GitLab) Commit: 3248f4e0f719af33c7cdf728bfe9624a70cd4ca9
Dependencies: Stopgaps:

Status badges

Description (last modified by vdelecroix)

  1. The argument order for the signature and documentation are reversed
    graphs.RandomRegular(d, n, seed=None)
    

and

   INPUT:

   * "n" - number of vertices

   * "d" - degree
  1. raise appropriate error when needed
  1. check input

Change History (4)

comment:1 Changed 3 years ago by vdelecroix

  • Authors set to Vincent Delecroix
  • Branch set to u/vdelecroix/25489
  • Commit set to 33af720de10d3c6a5fd473550536343b0995bf53
  • Description modified (diff)
  • Status changed from new to needs_review
  • Summary changed from argument order for graphs.RandomRegular to cleaning graphs.RandomRegular

New commits:

cbf02c925480: upgrade cryptominisat
33af72025489: cleaning graphs.RandomRegular

comment:2 Changed 3 years ago by git

  • Commit changed from 33af720de10d3c6a5fd473550536343b0995bf53 to 3248f4e0f719af33c7cdf728bfe9624a70cd4ca9

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

3248f4e25489: cleaning graphs.RandomRegular

comment:3 Changed 3 years ago by dcoudert

Why not changing the first """ to r""" and then latex notation `d` ? Both are certainly fine.

comment:4 Changed 3 years ago by chapoton

  • Status changed from needs_review to needs_work

does not apply

Note: See TracTickets for help on using tickets.