Skip to content

Commit 2e88ff0

Browse files
committed
use ._scaling_factor() instead of .formal()[1]
1 parent 2c86c69 commit 2e88ff0

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

src/sage/schemes/elliptic_curves/ell_curve_isogeny.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3346,7 +3346,7 @@ def dual(self):
33463346

33473347
# trac 7096
33483348
# this should take care of the case when the isogeny is not normalized.
3349-
u = self.formal(prec=2)[1]
3349+
u = self._scaling_factor()
33503350
isom = WeierstrassIsomorphism(E2pr, (u/F(d), 0, 0, 0))
33513351

33523352
E2 = isom.codomain()
@@ -3369,7 +3369,7 @@ def dual(self):
33693369
# the composition has the degree as a leading coefficient in
33703370
# the formal expansion.
33713371

3372-
phihat_sc = phi_hat.formal(prec=2)[1]
3372+
phihat_sc = phi_hat._scaling_factor()
33733373

33743374
sc = u * phihat_sc/F(d)
33753375

src/sage/schemes/elliptic_curves/hom.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -394,8 +394,7 @@ def is_normalized(self):
394394
sage: phi.is_normalized()
395395
True
396396
"""
397-
phi_formal = self.formal(prec=5)
398-
return phi_formal[1] == 1
397+
return self._scaling_factor() == 1
399398

400399

401400
def is_separable(self):

0 commit comments

Comments
 (0)