Skip to content

Commit 307712d

Browse files
author
Release Manager
committed
gh-36195: schemes/elliptic_curves: disable a test that takes too long A doctest introduced in #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: #36195 Reported by: Gonzalo Tornaría Reviewer(s):
2 parents 15a0566 + 902d91c commit 307712d

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/sage/schemes/elliptic_curves/ell_rational_field.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2306,7 +2306,7 @@ def gens(self, proof=None, **kwds):
23062306
sage: E.gens(algorithm="pari") # random output
23072307
[(5/4 : 5/8 : 1), (0 : 0 : 1)]
23082308
sage: E = EllipticCurve([0,2429469980725060,0,275130703388172136833647756388,0])
2309-
sage: len(E.gens(algorithm="pari"))
2309+
sage: len(E.gens(algorithm="pari")) # not tested (takes too long)
23102310
14
23112311
23122312
A non-integral example::

0 commit comments

Comments
 (0)