Skip to content

Commit

Permalink
sagemathgh-36195: schemes/elliptic_curves: disable a test that takes …
Browse files Browse the repository at this point in the history
…too long

    
A doctest introduced in sagemath#35626 takes ~50s on current hardware (for
comparision, doctesting the whole of
`src/sage/schemes/elliptic_curves/ell_rational_field.py` takes ~22s for
889 tests).

This is too long even for `--long`, but the example is still
interesting, so we just mark it `# not tested`.

### 📝 Checklist

- [x] The title is concise, informative, and self-explanatory.
- [x] The description explains in detail what this PR is about.
- [x] I have linked a relevant issue or discussion.


CC: @chriswuthrich (should be a trivial review 🙏)
    
URL: sagemath#36195
Reported by: Gonzalo Tornaría
Reviewer(s):
  • Loading branch information
Release Manager committed Sep 10, 2023
2 parents 15a0566 + 902d91c commit 307712d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/sage/schemes/elliptic_curves/ell_rational_field.py
Original file line number Diff line number Diff line change
Expand Up @@ -2306,7 +2306,7 @@ def gens(self, proof=None, **kwds):
sage: E.gens(algorithm="pari") # random output
[(5/4 : 5/8 : 1), (0 : 0 : 1)]
sage: E = EllipticCurve([0,2429469980725060,0,275130703388172136833647756388,0])
sage: len(E.gens(algorithm="pari"))
sage: len(E.gens(algorithm="pari")) # not tested (takes too long)
14
A non-integral example::
Expand Down

0 comments on commit 307712d

Please sign in to comment.