Skip to content

Commit 599fdd9

Browse files
author
Release Manager
committed
sagemathgh-41062: remove deprecations in finite_rings
after File: src/sage/rings/finite_rings/element_base.pyx ; PR: sagemath#32664 ; Closed Date: 2023-03-19 File: src/sage/rings/finite_rings/finite_field_constructor.py ; PR: sagemath#32664 ; Closed Date: 2023-03-19 File: src/sage/rings/finite_rings/finite_field_base.pyx ; PR: sagemath#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: sagemath#41062 Reported by: Frédéric Chapoton Reviewer(s):
2 parents b036c2b + b9a2e2a commit 599fdd9

File tree

5 files changed

+3
-88
lines changed

5 files changed

+3
-88
lines changed

build/pkgs/configure/checksums.ini

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
tarball=configure-VERSION.tar.gz
2-
sha1=22617a1a12a247801d8cbf658f7df9978034e1a9
3-
sha256=fae518a4a51179e16965cbc55eb40a9b6ee89db7d81b5f7d9d0466262611b8a1
2+
sha1=32d7acb01f87cb27d4eba3306415c989a503eead
3+
sha256=16cd4131bf29641c784a9feb738e9f06acdbd0cea1a4971c057da7fa4d2cbfc7
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
ad729a3fe96044ea67fc1f78ea2e0bf2706f29eb
1+
e2e3770e034620a429291b30e676022d3c19a7c7

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
@@ -2224,30 +2224,3 @@ def unpickle_FiniteField_prm(_type, order, variable_name, kwargs):
22242224

22252225
register_unpickle_override(
22262226
'sage.rings.ring', 'unpickle_FiniteField_prm', unpickle_FiniteField_prm)
2227-
2228-
2229-
def is_FiniteField(R):
2230-
r"""
2231-
Return whether the implementation of ``R`` has the interface provided by
2232-
the standard finite field implementation.
2233-
2234-
This function is deprecated.
2235-
2236-
EXAMPLES::
2237-
2238-
sage: from sage.rings.finite_rings.finite_field_base import is_FiniteField
2239-
sage: is_FiniteField(GF(9,'a'))
2240-
doctest:...: DeprecationWarning: the function is_FiniteField is deprecated; use isinstance(x, sage.rings.finite_rings.finite_field_base.FiniteField) instead
2241-
See https://github.com/sagemath/sage/issues/32664 for details.
2242-
True
2243-
sage: is_FiniteField(GF(next_prime(10^10)))
2244-
True
2245-
2246-
Note that the integers modulo n are not backed by the finite field type::
2247-
2248-
sage: is_FiniteField(Integers(7))
2249-
False
2250-
"""
2251-
from sage.misc.superseded import deprecation
2252-
deprecation(32664, "the function is_FiniteField is deprecated; use isinstance(x, sage.rings.finite_rings.finite_field_base.FiniteField) instead")
2253-
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.
@@ -839,34 +835,4 @@ def create_object(self, version, key, **kwds):
839835
GF = FiniteField = FiniteFieldFactory("FiniteField")
840836

841837

842-
def is_PrimeFiniteField(x):
843-
"""
844-
Return ``True`` if ``x`` is a prime finite field.
845-
846-
This function is deprecated.
847-
848-
EXAMPLES::
849-
850-
sage: from sage.rings.finite_rings.finite_field_constructor import is_PrimeFiniteField
851-
sage: is_PrimeFiniteField(QQ)
852-
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
853-
See https://github.com/sagemath/sage/issues/32664 for details.
854-
False
855-
sage: is_PrimeFiniteField(GF(7))
856-
True
857-
sage: is_PrimeFiniteField(GF(7^2, 'a'))
858-
False
859-
sage: is_PrimeFiniteField(GF(next_prime(10^90, proof=False)))
860-
True
861-
"""
862-
from sage.misc.superseded import deprecation
863-
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")
864-
865-
from .finite_field_prime_modn import FiniteField_prime_modn
866-
from sage.rings.finite_rings.finite_field_base import FiniteField as FiniteField_generic
867-
868-
return isinstance(x, FiniteField_prime_modn) or \
869-
(isinstance(x, FiniteField_generic) and x.degree() == 1)
870-
871-
872838
zech_log_bound = 2**16

0 commit comments

Comments
 (0)