Skip to content

Commit 1329316

Browse files
author
Release Manager
committed
gh-36011: Regular sequence: closing experimental phase of regular sequence module This PR closes the experimental-phase of the regular sequences module. This module has been experimental für a very long time now. Lately #35894 has been completed so this is now a fully-functional ring/algebra. Except for the consequences of #35894 (renaming; see below), the interface is stable for very long time. This PR does: - Rename to `RegularSequenceRing` (as this structure now is a ring due to #35894). - Remove the experimental warning. ### 📝 Checklist - [x] The title is concise, informative, and self-explanatory. - [x] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [x] I have created tests covering the changes. - [x] I have updated the documentation accordingly. ### ⌛ Dependencies - #35894 URL: #36011 Reported by: Daniel Krenn Reviewer(s): cheuberg, Daniel Krenn
2 parents c4b3f0d + cd04076 commit 1329316

File tree

4 files changed

+139
-170
lines changed

4 files changed

+139
-170
lines changed

src/doc/en/reference/combinat/module_list.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -143,7 +143,6 @@ Comprehensive Module List
143143
sage/combinat/integer_vector_weighted
144144
sage/combinat/integer_vectors_mod_permgroup
145145
sage/combinat/interval_posets
146-
sage/combinat/k_regular_sequence
147146
sage/combinat/k_tableau
148147
sage/combinat/kazhdan_lusztig
149148
sage/combinat/key_polynomial
@@ -206,6 +205,7 @@ Comprehensive Module List
206205
sage/combinat/quickref
207206
sage/combinat/ranker
208207
sage/combinat/recognizable_series
208+
sage/combinat/regular_sequence
209209
sage/combinat/restricted_growth
210210
sage/combinat/ribbon
211211
sage/combinat/ribbon_shaped_tableau

src/sage/combinat/all.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -277,7 +277,7 @@
277277
lazy_import('sage.combinat.binary_recurrence_sequences',
278278
'BinaryRecurrenceSequence')
279279
lazy_import('sage.combinat.recognizable_series', 'RecognizableSeriesSpace')
280-
lazy_import('sage.combinat.k_regular_sequence', 'kRegularSequenceSpace')
280+
lazy_import('sage.combinat.regular_sequence', 'RegularSequenceRing')
281281

282282
# Six Vertex Model
283283
lazy_import('sage.combinat.six_vertex_model', 'SixVertexModel')

src/sage/combinat/recognizable_series.py

Lines changed: 6 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -27,27 +27,13 @@
2727
In particular, minimization is called before checking if a series is
2828
nonzero.
2929
30-
.. WARNING::
31-
32-
As this code is experimental, warnings are thrown when a
33-
recognizable series space is created for the first time in a
34-
session (see :class:`sage.misc.superseded.experimental`).
35-
36-
TESTS::
37-
38-
sage: Rec = RecognizableSeriesSpace(ZZ, [0, 1])
39-
doctest:...: FutureWarning: This class/method/function is
40-
marked as experimental. It, its functionality or its interface
41-
might change without a formal deprecation.
42-
See https://github.com/sagemath/sage/issues/21202 for details.
43-
4430
4531
Various
4632
=======
4733
4834
.. SEEALSO::
4935
50-
:mod:`k-regular sequence <sage.combinat.k_regular_sequence>`,
36+
:mod:`k-regular sequence <sage.combinat.regular_sequence>`,
5137
:mod:`sage.rings.cfinite_sequence`,
5238
:mod:`sage.combinat.binary_recurrence_sequences`.
5339
@@ -79,7 +65,6 @@
7965
from functools import wraps
8066

8167
from sage.misc.cachefunc import cached_method
82-
from sage.misc.superseded import experimental
8368
from sage.structure.element import ModuleElement
8469
from sage.structure.parent import Parent
8570
from sage.structure.unique_representation import UniqueRepresentation
@@ -1297,7 +1282,7 @@ def _add_(self, other):
12971282
12981283
EXAMPLES::
12991284
1300-
sage: Seq2 = kRegularSequenceSpace(2, ZZ)
1285+
sage: Seq2 = RegularSequenceRing(2, ZZ)
13011286
sage: E = Seq2((Matrix([[0, 1], [0, 1]]), Matrix([[0, 0], [0, 1]])),
13021287
....: vector([1, 0]), vector([1, 1]))
13031288
sage: E
@@ -1336,7 +1321,7 @@ def _neg_(self):
13361321
13371322
EXAMPLES::
13381323
1339-
sage: Seq2 = kRegularSequenceSpace(2, ZZ)
1324+
sage: Seq2 = RegularSequenceRing(2, ZZ)
13401325
sage: E = Seq2((Matrix([[0, 1], [0, 1]]), Matrix([[0, 0], [0, 1]])),
13411326
....: vector([1, 0]), vector([1, 1]))
13421327
sage: -E
@@ -1363,7 +1348,7 @@ def _rmul_(self, other):
13631348
13641349
EXAMPLES::
13651350
1366-
sage: Seq2 = kRegularSequenceSpace(2, ZZ)
1351+
sage: Seq2 = RegularSequenceRing(2, ZZ)
13671352
sage: E = Seq2((Matrix([[0, 1], [0, 1]]), Matrix([[0, 0], [0, 1]])),
13681353
....: vector([1, 0]), vector([1, 1]))
13691354
sage: M = 2 * E # indirect doctest
@@ -1427,7 +1412,7 @@ def _lmul_(self, other):
14271412
14281413
EXAMPLES::
14291414
1430-
sage: Seq2 = kRegularSequenceSpace(2, ZZ)
1415+
sage: Seq2 = RegularSequenceRing(2, ZZ)
14311416
sage: E = Seq2((Matrix([[0, 1], [0, 1]]), Matrix([[0, 0], [0, 1]])),
14321417
....: vector([1, 0]), vector([1, 1]))
14331418
sage: M = E * 2 # indirect doctest
@@ -1497,7 +1482,7 @@ def hadamard_product(self, other):
14971482
14981483
EXAMPLES::
14991484
1500-
sage: Seq2 = kRegularSequenceSpace(2, ZZ)
1485+
sage: Seq2 = RegularSequenceRing(2, ZZ)
15011486
15021487
sage: E = Seq2((Matrix([[0, 1], [0, 1]]), Matrix([[0, 0], [0, 1]])),
15031488
....: vector([1, 0]), vector([1, 1]))
@@ -1741,7 +1726,6 @@ def __normalize__(cls,
17411726

17421727
return (coefficient_ring, indices, category, minimize_results)
17431728

1744-
@experimental(issue_number=21202)
17451729
def __init__(self, coefficient_ring, indices, category, minimize_results):
17461730
r"""
17471731
See :class:`RecognizableSeriesSpace` for details.

0 commit comments

Comments
 (0)