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.

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

Random failure in number_field_ideal_rel.py #30270

Closed
vbraun opened this issue Aug 2, 2020 · 9 comments
Closed

Random failure in number_field_ideal_rel.py #30270

vbraun opened this issue Aug 2, 2020 · 9 comments

Comments

@vbraun
Copy link
Member

vbraun commented Aug 2, 2020

**********************************************************************
File "src/sage/rings/number_field/number_field_ideal_rel.py", line 509, in sage.rings.number_field.number_field_ideal_rel.NumberFieldFractionalIdeal_rel.ideal_below
Failed example:
    J = I.ideal_below(); J
Expected:
    Fractional ideal (-b)
Got:
    Fractional ideal (b)
**********************************************************************
1 item had failures:
   1 of  48 in sage.rings.number_field.number_field_ideal_rel.NumberFieldFractionalIdeal_rel.ideal_below
    [232 tests, 1 failure, 0.59 s]
----------------------------------------------------------------------
sage -t --long --random-seed=0 src/sage/rings/number_field/number_field_ideal_rel.py  # 1 doctest failed
----------------------------------------------------------------------

CC: @loefflerd

Component: number fields

Keywords: random_fail

Author: Samuel Lelièvre

Branch/Commit: 4a0c242

Reviewer: Matthias Koeppe

Issue created by migration from https://trac.sagemath.org/ticket/30270

@vbraun vbraun added this to the sage-9.2 milestone Aug 2, 2020
@slel
Copy link
Member

slel commented Aug 3, 2020

Author: Samuel Lelièvre

@slel
Copy link
Member

slel commented Aug 3, 2020

New commits:

1815cd8#30270 Fix number_field_ideal_rel doctest

@slel
Copy link
Member

slel commented Aug 3, 2020

Commit: 1815cd8

@slel
Copy link
Member

slel commented Aug 3, 2020

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 3, 2020

Changed commit from 1815cd8 to 4a0c242

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 3, 2020

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

4a0c242#30270 Fix number_field_ideal_rel doctest

@slel
Copy link
Member

slel commented Aug 3, 2020

comment:3

Sorry, I had a doubled prompt sage: sage:. I force pushed a fixed version.

@mkoeppe
Copy link
Member

mkoeppe commented Aug 5, 2020

Reviewer: Matthias Koeppe

@vbraun
Copy link
Member Author

vbraun commented Aug 9, 2020

Changed branch from public/30270-fix-number-field-ideal-rel-doctest to 4a0c242

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants