From 8d090449049b4d8c0de42e69edb540147a0c191a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?R=C3=A9my=20Oudompheng?= Date: Mon, 15 May 2023 11:36:10 +0200 Subject: [PATCH] Avoid redundant computations in finite field .conjugate() method The previous implementation was factoring the order (characteristic^degree) to obtain the field degree. --- src/sage/rings/finite_rings/element_base.pyx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/sage/rings/finite_rings/element_base.pyx b/src/sage/rings/finite_rings/element_base.pyx index a7e32004002..6707a0535f4 100755 --- a/src/sage/rings/finite_rings/element_base.pyx +++ b/src/sage/rings/finite_rings/element_base.pyx @@ -1006,7 +1006,7 @@ cdef class FinitePolyExtElement(FiniteRingElement): sage: G32(m1) == g1 True """ - [(p, k2)] = list(self.parent().cardinality().factor()) + k2 = self.parent().degree() if k2 % 2: raise TypeError("cardinality of the field must be a square number") k = k2 / 2