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 ell_rational_field.py #17415

Closed
jdemeyer opened this issue Nov 28, 2014 · 5 comments
Closed

Random failure in ell_rational_field.py #17415

jdemeyer opened this issue Nov 28, 2014 · 5 comments

Comments

@jdemeyer
Copy link

Garbage collection coming at the wrong time can cause

File "src/sage/schemes/elliptic_curves/ell_rational_field.py", line 590, in sage.schemes.elliptic_curves.ell_rational_field.EllipticCurve_rational_field.pari_curve
Failed example:
    _ = E.__dict__.pop('_pari_curve')  # clear cached data
Exception raised:
    Traceback (most recent call last):
      File "/usr/local/src/sage-config/local/lib/python2.7/site-packages/sage/doctest/forker.py", line 488, in _run
        self.compile_and_execute(example, compiler, test.globs)
      File "/usr/local/src/sage-config/local/lib/python2.7/site-packages/sage/doctest/forker.py", line 851, in compile_and_execute
        exec(compiled, globs)
      File "<doctest sage.schemes.elliptic_curves.ell_rational_field.EllipticCurve_rational_field.pari_curve[9]>", line 1, in <module>
        _ = E.__dict__.pop('_pari_curve')  # clear cached data
    KeyError: '_pari_curve'

CC: @pjbruin @vbraun

Component: elliptic curves

Keywords: random_fail

Author: Peter Bruin

Branch/Commit: 6247969

Reviewer: Volker Braun

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

@jdemeyer jdemeyer added this to the sage-6.5 milestone Nov 28, 2014
@pjbruin
Copy link
Contributor

pjbruin commented Nov 28, 2014

Branch: u/pbruin/17415-ell_random_fail

@pjbruin
Copy link
Contributor

pjbruin commented Nov 28, 2014

Commit: 6247969

@pjbruin
Copy link
Contributor

pjbruin commented Nov 28, 2014

Author: Peter Bruin

@vbraun
Copy link
Member

vbraun commented Nov 28, 2014

Reviewer: Volker Braun

@vbraun
Copy link
Member

vbraun commented Nov 29, 2014

Changed branch from u/pbruin/17415-ell_random_fail to 6247969

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