Skip to content

Commit f5c04cb

Browse files
author
Release Manager
committed
gh-41062: remove deprecations in finite_rings after File: src/sage/rings/finite_rings/element_base.pyx ; PR: #32664 ; Closed Date: 2023-03-19 File: src/sage/rings/finite_rings/finite_field_constructor.py ; PR: #32664 ; Closed Date: 2023-03-19 File: src/sage/rings/finite_rings/finite_field_base.pyx ; PR: #32664 ; Closed Date: 2023-03-19 ### 📝 Checklist - [x] The title is concise and informative. - [x] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. URL: #41062 Reported by: Frédéric Chapoton Reviewer(s):
2 parents 8c8b1ef + b9a2e2a commit f5c04cb

File tree

3 files changed

+0
-85
lines changed

3 files changed

+0
-85
lines changed

src/sage/rings/finite_rings/element_base.pyx

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -24,30 +24,6 @@ from sage.rings.integer_ring import ZZ
2424
from sage.rings.integer import Integer
2525

2626

27-
def is_FiniteFieldElement(x):
28-
"""
29-
Return ``True`` if ``x`` is a finite field element.
30-
31-
This function is deprecated.
32-
33-
EXAMPLES::
34-
35-
sage: from sage.rings.finite_rings.element_base import is_FiniteFieldElement
36-
sage: is_FiniteFieldElement(1)
37-
doctest:...: DeprecationWarning: the function is_FiniteFieldElement is deprecated; use isinstance(x, sage.structure.element.FieldElement) and x.parent().is_finite() instead
38-
See https://github.com/sagemath/sage/issues/32664 for details.
39-
False
40-
sage: is_FiniteFieldElement(IntegerRing())
41-
False
42-
sage: is_FiniteFieldElement(GF(5)(2))
43-
True
44-
"""
45-
from sage.misc.superseded import deprecation
46-
deprecation(32664, "the function is_FiniteFieldElement is deprecated; use isinstance(x, sage.structure.element.FieldElement) and x.parent().is_finite() instead")
47-
48-
from sage.rings.finite_rings.finite_field_base import FiniteField
49-
return isinstance(x, Element) and isinstance(x.parent(), FiniteField)
50-
5127

5228
cdef class FiniteRingElement(CommutativeRingElement):
5329
def _nth_root_common(self, n, all, algorithm, cunningham):

src/sage/rings/finite_rings/finite_field_base.pyx

Lines changed: 0 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -2239,30 +2239,3 @@ def unpickle_FiniteField_prm(_type, order, variable_name, kwargs):
22392239

22402240
register_unpickle_override(
22412241
'sage.rings.ring', 'unpickle_FiniteField_prm', unpickle_FiniteField_prm)
2242-
2243-
2244-
def is_FiniteField(R):
2245-
r"""
2246-
Return whether the implementation of ``R`` has the interface provided by
2247-
the standard finite field implementation.
2248-
2249-
This function is deprecated.
2250-
2251-
EXAMPLES::
2252-
2253-
sage: from sage.rings.finite_rings.finite_field_base import is_FiniteField
2254-
sage: is_FiniteField(GF(9,'a'))
2255-
doctest:...: DeprecationWarning: the function is_FiniteField is deprecated; use isinstance(x, sage.rings.finite_rings.finite_field_base.FiniteField) instead
2256-
See https://github.com/sagemath/sage/issues/32664 for details.
2257-
True
2258-
sage: is_FiniteField(GF(next_prime(10^10)))
2259-
True
2260-
2261-
Note that the integers modulo n are not backed by the finite field type::
2262-
2263-
sage: is_FiniteField(Integers(7))
2264-
False
2265-
"""
2266-
from sage.misc.superseded import deprecation
2267-
deprecation(32664, "the function is_FiniteField is deprecated; use isinstance(x, sage.rings.finite_rings.finite_field_base.FiniteField) instead")
2268-
return isinstance(R, FiniteField)

src/sage/rings/finite_rings/finite_field_constructor.py

Lines changed: 0 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -178,10 +178,6 @@
178178
from sage.rings.polynomial.polynomial_element import Polynomial
179179
from sage.rings.integer import Integer
180180

181-
# the import below is just a redirection
182-
from sage.rings.finite_rings.finite_field_base import is_FiniteField
183-
assert is_FiniteField # just to silent pyflakes
184-
185181
try:
186182
# We don't late import this because this means trouble with the Givaro library
187183
# On a Macbook Pro OSX 10.5.8, this manifests as a Bus Error on exiting Sage.
@@ -868,34 +864,4 @@ def create_object(self, version, key, **kwds):
868864
GF = FiniteField = FiniteFieldFactory("FiniteField")
869865

870866

871-
def is_PrimeFiniteField(x):
872-
"""
873-
Return ``True`` if ``x`` is a prime finite field.
874-
875-
This function is deprecated.
876-
877-
EXAMPLES::
878-
879-
sage: from sage.rings.finite_rings.finite_field_constructor import is_PrimeFiniteField
880-
sage: is_PrimeFiniteField(QQ)
881-
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
882-
See https://github.com/sagemath/sage/issues/32664 for details.
883-
False
884-
sage: is_PrimeFiniteField(GF(7))
885-
True
886-
sage: is_PrimeFiniteField(GF(7^2, 'a'))
887-
False
888-
sage: is_PrimeFiniteField(GF(next_prime(10^90, proof=False)))
889-
True
890-
"""
891-
from sage.misc.superseded import deprecation
892-
deprecation(32664, "the function is_PrimeFiniteField is deprecated; use isinstance(x, sage.rings.finite_rings.finite_field_base.FiniteField) and x.is_prime_field() instead")
893-
894-
from .finite_field_prime_modn import FiniteField_prime_modn
895-
from sage.rings.finite_rings.finite_field_base import FiniteField as FiniteField_generic
896-
897-
return isinstance(x, FiniteField_prime_modn) or \
898-
(isinstance(x, FiniteField_generic) and x.degree() == 1)
899-
900-
901867
zech_log_bound = 2**16

0 commit comments

Comments
 (0)