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

Commit 106b22e

Browse files
author
Matthias Koeppe
committed
Fixups
1 parent b82fb1e commit 106b22e

File tree

5 files changed

+11
-3
lines changed

5 files changed

+11
-3
lines changed

src/sage/modules/free_module.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -183,7 +183,6 @@
183183
import sage.misc.latex as latex
184184

185185
from sage.modules.module import Module
186-
import sage.rings.finite_rings.finite_field_constructor as finite_field
187186
import sage.rings.ring as ring
188187
import sage.rings.abc
189188
import sage.rings.integer_ring
@@ -195,6 +194,7 @@
195194
from sage.categories.infinite_enumerated_sets import InfiniteEnumeratedSets
196195
from sage.misc.lazy_attribute import lazy_attribute
197196
from sage.misc.randstate import current_randstate
197+
from sage.rings.finite_rings.finite_field_base import FiniteField
198198
from sage.structure.factory import UniqueFactory
199199
from sage.structure.sequence import Sequence
200200
from sage.structure.richcmp import (richcmp_method, rich_to_bool, richcmp,
@@ -6339,7 +6339,7 @@ def _element_constructor_(self, e, *args, **kwds):
63396339
"""
63406340
try:
63416341
k = e.parent()
6342-
if finite_field.isinstance(k, FiniteField) and k.base_ring() == self.base_ring() and k.degree() == self.degree():
6342+
if isinstance(k, FiniteField) and k.base_ring() == self.base_ring() and k.degree() == self.degree():
63436343
return self(e._vector_())
63446344
except AttributeError:
63456345
pass

src/sage/rings/finite_rings/element_base.pyx

+1
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ def is_FiniteFieldElement(x):
3232
sage: from sage.rings.finite_rings.element_base import is_FiniteFieldElement
3333
sage: is_FiniteFieldElement(1)
3434
doctest:...: DeprecationWarning: the function is_FiniteFieldElement is deprecated; use isinstance(x, sage.structure.element.FieldElement) and x.parent().is_finite() instead
35+
See https://trac.sagemath.org/32664 for details.
3536
False
3637
sage: is_FiniteFieldElement(IntegerRing())
3738
False

src/sage/rings/finite_rings/finite_field_base.pyx

+1
Original file line numberDiff line numberDiff line change
@@ -2112,6 +2112,7 @@ def is_FiniteField(R):
21122112
sage: from sage.rings.finite_rings.finite_field_base import FiniteField
21132113
sage: is_FiniteField(GF(9,'a'))
21142114
doctest:...: DeprecationWarning: the function is_FiniteField is deprecated; use isinstance(x, sage.rings.finite_rings.finite_field_base.FiniteField) instead
2115+
See https://trac.sagemath.org/32664 for details.
21152116
True
21162117
sage: is_FiniteField(GF(next_prime(10^10)))
21172118
True

src/sage/rings/finite_rings/finite_field_constructor.py

+2
Original file line numberDiff line numberDiff line change
@@ -797,8 +797,10 @@ def is_PrimeFiniteField(x):
797797
798798
EXAMPLES::
799799
800+
sage: from sage.rings.finite_rings.finite_field_constructor import is_PrimeFiniteField
800801
sage: is_PrimeFiniteField(QQ)
801802
doctest:...: DeprecationWarning: the function is_PrimeFiniteField is deprecated; use isinstance(x, sage.rings.finite_rings.finite_field_base.FiniteField) and x.is_prime_field() instead
803+
See https://trac.sagemath.org/32664 for details.
802804
False
803805
sage: is_PrimeFiniteField(GF(7))
804806
True

src/sage/schemes/hyperelliptic_curves/constructor.py

+5-1
Original file line numberDiff line numberDiff line change
@@ -249,7 +249,11 @@ def HyperellipticCurve(f, h=0, names=None, PP=None, check_squarefree=True):
249249

250250
genus_classes = {2: HyperellipticCurve_g2}
251251

252-
is_pAdicField = lambda x: isinstance(x, sage.rings.abc.pAdicField)
252+
def is_FiniteField(x):
253+
return isinstance(x, FiniteField)
254+
255+
def is_pAdicField(x):
256+
return isinstance(x, sage.rings.abc.pAdicField)
253257

254258
fields = [
255259
("FiniteField", is_FiniteField, HyperellipticCurve_finite_field),

0 commit comments

Comments
 (0)