Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove randomly failing doctest in src/sage/graphs/generators/random.py #34520

Closed
dcoudert opened this issue Sep 11, 2022 · 7 comments
Closed

Remove randomly failing doctest in src/sage/graphs/generators/random.py #34520

dcoudert opened this issue Sep 11, 2022 · 7 comments
Labels
c: graph theory p: minor / 4 t: enhancement
Milestone
sage-9.8

Comments

@dcoudert
Copy link
Contributor 

sage -t --random-seed=124699541928353137721504742146487601068 src/sage/graphs/generators/random.py
**********************************************************************
File "src/sage/graphs/generators/random.py", line 817, in sage.graphs.generators.random.RandomHolmeKim
Failed example:
    G.subgraph_search(C3)
Expected:
    Subgraph of (): Graph on 3 vertices
Got:
    <BLANKLINE>
**********************************************************************
1 item had failures:
   1 of   6 in sage.graphs.generators.random.RandomHolmeKim
    [197 tests, 1 failure, 1.92 s]

See also #32544 comment:33

Component: graph theory

Author: David Coudert

Branch/Commit: 1f4fdcd

Reviewer: Frédéric Chapoton

Issue created by migration from https://trac.sagemath.org/ticket/34520
@dcoudert dcoudert added this to the sage-9.8 milestone Sep 11, 2022
@dcoudert
Copy link
Contributor Author

Branch: public/graphs/34520

@dcoudert
Copy link
Contributor Author

New commits:

1f4fdcdtrac #34520: remove doctest in RandomHolmeKim generator

@dcoudert
Copy link
Contributor Author

Commit: 1f4fdcd

@fchapoton
Copy link
Contributor

Reviewer: Frédéric Chapoton

@fchapoton
Copy link
Contributor

comment:2

ok

@DaveWitteMorris

This comment has been minimized.

Sign in to view
@vbraun
Copy link
Member

vbraun commented Sep 27, 2022

Changed branch from public/graphs/34520 to 1f4fdcd

@vbraun vbraun closed this as completed in 12890b6 Sep 27, 2022
@kliem kliem mentioned this issue Jan 29, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: graph theory p: minor / 4 t: enhancement
Projects
None yet
Milestone
sage-9.8
Development

No branches or pull requests
4 participants
@vbraun @fchapoton @dcoudert @DaveWitteMorris