Skip to content

Commit 2e84eda

Browse files
committed
Fix doctest for multiplication_by_m
The current doctest can create a singular curve, for example with `random-seed=129979402993014347526756638208394445987`.
1 parent 30b3d78 commit 2e84eda

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

src/sage/schemes/elliptic_curves/ell_generic.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2448,8 +2448,10 @@ def multiplication_by_m(self, m, x_only=False):
24482448
::
24492449
24502450
sage: p = 7
2451-
sage: K.<a> = GF(p^2)
2452-
sage: E = EllipticCurve(K, [K.random_element(), K.random_element()])
2451+
sage: K.<i> = GF(p^2)
2452+
sage: a, b = K.random_element(), K.random_element()
2453+
sage: while 4*a^3 + 27*b^2 == 0: a, b = K.random_element(), K.random_element()
2454+
sage: E = EllipticCurve(K, a, b)
24532455
sage: E.multiplication_by_m(p * 2)[0] == E.multiplication_by_m(p * 2, x_only=True)
24542456
True
24552457
"""

0 commit comments

Comments
 (0)