Skip to content
This repository has been archived by the owner on Jan 30, 2023. It is now read-only.

Commit

Permalink
remove experimental warnings for composite elliptic-curve isogenies
Browse files Browse the repository at this point in the history
  • Loading branch information
yyyyx4 committed Aug 22, 2022
1 parent 7f71494 commit fd50425
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 17 deletions.
6 changes: 2 additions & 4 deletions src/sage/schemes/elliptic_curves/ell_field.py
Original file line number Diff line number Diff line change
Expand Up @@ -1090,7 +1090,7 @@ def isogeny(self, kernel, codomain=None, degree=None, model=None, check=True, al
Kohel's algorithm is currently only implemented for cyclic
isogenies, with the exception of `[2]`.
- Factored Isogenies (*experimental* --- see
- Factored Isogenies (see
:mod:`~sage.schemes.elliptic_curves.hom_composite`):
Given a list of points which generate a composite-order
subgroup, decomposes the isogeny into prime-degree steps.
Expand Down Expand Up @@ -1177,9 +1177,7 @@ def isogeny(self, kernel, codomain=None, degree=None, model=None, check=True, al
sage: E = EllipticCurve(GF(2^32-5), [170246996, 2036646110])
sage: P = E.lift_x(2)
sage: E.isogeny(P, algorithm="factored") # experimental
doctest:warning
...
sage: E.isogeny(P, algorithm="factored")
Composite morphism of degree 1073721825 = 3^4*5^2*11*19*43*59:
From: Elliptic Curve defined by y^2 = x^3 + 170246996*x + 2036646110 over Finite Field of size 4294967291
To: Elliptic Curve defined by y^2 = x^3 + 272790262*x + 1903695400 over Finite Field of size 4294967291
Expand Down
1 change: 0 additions & 1 deletion src/sage/schemes/elliptic_curves/hom.py
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,6 @@ def degree(self):
is the product of the degrees of the individual factors::
sage: from sage.schemes.elliptic_curves.hom_composite import EllipticCurveHom_composite
doctest:warning ...
sage: E = EllipticCurve(GF(419), [1,0])
sage: P, = E.gens()
sage: phi = EllipticCurveHom_composite(E, P+P)
Expand Down
13 changes: 1 addition & 12 deletions src/sage/schemes/elliptic_curves/hom_composite.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,21 +7,13 @@
while exposing (close to) the same interface as "normal", unfactored
elliptic-curve isogenies.
.. WARNING::
This module is currently considered experimental.
It may change in a future release without prior warning, or even
be removed altogether if things turn out to be unfixably broken.
EXAMPLES:
The following example would take quite literally forever with the
straightforward :class:`EllipticCurveIsogeny` implementation, but
decomposing into prime steps is exponentially faster::
sage: from sage.schemes.elliptic_curves.hom_composite import EllipticCurveHom_composite
doctest:warning
...
sage: p = 3 * 2^143 - 1
sage: GF(p^2).inject_variables()
Defining z2
Expand Down Expand Up @@ -95,9 +87,6 @@
from sage.schemes.elliptic_curves.ell_curve_isogeny import EllipticCurveIsogeny
from sage.schemes.elliptic_curves.weierstrass_morphism import WeierstrassIsomorphism

from sage.misc.superseded import experimental_warning
experimental_warning(32744, 'EllipticCurveHom_composite is experimental code.')

#TODO: implement sparse strategies? (cf. the SIKE cryptosystem)

def _eval_factored_isogeny(phis, P):
Expand Down Expand Up @@ -828,7 +817,7 @@ def make_default():
This method exists only temporarily to make testing more
convenient while :class:`EllipticCurveHom_composite` is
experimental.
not yet the default.
EXAMPLES::
Expand Down

0 comments on commit fd50425

Please sign in to comment.