Skip to content

Commit

Permalink
apply review changes
Browse files Browse the repository at this point in the history
  • Loading branch information
grhkm21 committed Sep 11, 2024
1 parent 228a2bf commit 163191f
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion src/sage/schemes/elliptic_curves/ell_finite_field.py
Original file line number Diff line number Diff line change
Expand Up @@ -2863,6 +2863,8 @@ def find_q(m, m4_fac, D):
continue
# This tests whether the curve has given order
if Et.has_order(m_val):
# TODO: remove after 38617
Et.set_order(m_val, check=False)
seen.add(Et)
yield Et

Expand Down Expand Up @@ -3167,8 +3169,10 @@ def abs_products_under(bound):
E = EllipticCurve(K, j=j0)
# `E.twists()` also contains E.
for Et in E.twists():
# `num_checks=1` is sufficient for prime order.
# `num_checks=1` is sufficient for prime order
if Et.has_order(N, num_checks=1):
# TODO: remove after 38617
Et.set_order(N, check=False)
yield Et

if p != 1:
Expand Down

0 comments on commit 163191f

Please sign in to comment.