Opened 5 years ago

Closed 5 years ago

#23013 closed defect (fixed)

Random failure in libecm.pyx

Reported by: vbraun Owned by:
Priority: blocker Milestone: sage-8.0
Component: packages: standard Keywords: random_fail
Cc: jdemeyer Merged in:
Authors: Volker Braun Reviewers: Travis Scrimshaw
Report Upstream: N/A Work issues:
Branch: 092dc4d (Commits, GitHub, GitLab) Commit: 092dc4da1176197c97ca623753d84ce44bfc4315
Dependencies: Stopgaps:

Status badges

Description

sage -t --long src/sage/libs/libecm.pyx
**********************************************************************
File "src/sage/libs/libecm.pyx", line 22, in sage.libs.libecm
Failed example:
    result[1] in [3, 9, 27, 37, 999] or result[1]
Expected:
    True
Got:
    111
**********************************************************************
1 item had failures:
   1 of   9 in sage.libs.libecm
    [28 tests, 1 failure, 1.94 s]

Change History (7)

comment:1 Changed 5 years ago by zimmerma

since 111 and 333 are missing divisors of 999, it should be:

result[1] in [3, 9, 27, 37, 111, 333, 999] or result[1]

comment:2 Changed 5 years ago by jdemeyer

  • Priority changed from major to blocker

comment:3 Changed 5 years ago by vbraun

  • Authors set to Volker Braun
  • Status changed from new to needs_review

comment:4 Changed 5 years ago by tscrim

Branch?

comment:5 Changed 5 years ago by vbraun

  • Branch set to u/vbraun/random_failure_in_libecm_pyx

comment:6 Changed 5 years ago by tscrim

  • Commit set to 092dc4da1176197c97ca623753d84ce44bfc4315
  • Reviewers set to Travis Scrimshaw
  • Status changed from needs_review to positive_review

LGTM.


New commits:

092dc4dAdd the missing divisors of 999 in doctest

comment:7 Changed 5 years ago by vbraun

  • Branch changed from u/vbraun/random_failure_in_libecm_pyx to 092dc4da1176197c97ca623753d84ce44bfc4315
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.