Skip to content

Commit 40d0e60

Browse files
committed
using Integer more
1 parent bb8b646 commit 40d0e60

File tree

5 files changed

+24
-23
lines changed

5 files changed

+24
-23
lines changed

src/sage/combinat/affine_permutation.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
from sage.misc.lazy_import import lazy_import
2323
from sage.misc.misc_c import prod
2424
from sage.misc.prandom import randint
25-
from sage.rings.integer_ring import ZZ
25+
from sage.rings.integer import Integer
2626
from sage.structure.list_clone import ClonableArray
2727
from sage.structure.parent import Parent
2828
from sage.structure.unique_representation import UniqueRepresentation
@@ -75,8 +75,8 @@ def __init__(self, parent, lst, check=True):
7575
Type A affine permutation with window [1, 2, 3, 4]
7676
"""
7777
if check:
78-
lst = [ZZ(val) for val in lst]
79-
self.k = ZZ(parent.k)
78+
lst = [Integer(val) for val in lst]
79+
self.k = parent.k
8080
self.n = self.k + 1
8181
# This N doesn't matter for type A, but comes up in all other types.
8282
if parent.cartan_type()[0] == 'A':
@@ -2014,7 +2014,7 @@ def __init__(self, cartan_type):
20142014
"""
20152015
Parent.__init__(self, category=AffineWeylGroups())
20162016
ct = CartanType(cartan_type)
2017-
self.k = ZZ(ct.n)
2017+
self.k = Integer(ct.n)
20182018
self.n = ct.rank()
20192019
# This N doesn't matter for type A, but comes up in all other types.
20202020
if ct.letter == 'A':

src/sage/combinat/baxter_permutations.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
Baxter permutations
33
"""
44
from sage.combinat.permutation import Permutations
5+
from sage.rings.integer import Integer
56
from sage.rings.integer_ring import ZZ
67
from sage.sets.disjoint_union_enumerated_sets import DisjointUnionEnumeratedSets
78
from sage.structure.parent import Parent
@@ -77,11 +78,11 @@ def __init__(self, n):
7778
Baxter permutations of size 5
7879
"""
7980
self.element_class = Permutations(n).element_class
80-
self._n = ZZ(n)
81+
self._n = Integer(n)
8182
from sage.categories.finite_enumerated_sets import FiniteEnumeratedSets
8283
super().__init__(category=FiniteEnumeratedSets())
8384

84-
def _repr_(self):
85+
def _repr_(self) -> str:
8586
"""
8687
Return a string representation of ``self``.
8788
@@ -91,9 +92,9 @@ def _repr_(self):
9192
sage: BaxterPermutations_size(5)
9293
Baxter permutations of size 5
9394
"""
94-
return "Baxter permutations of size %s" % self._n
95+
return f"Baxter permutations of size {self._n}"
9596

96-
def __contains__(self, x):
97+
def __contains__(self, x) -> bool:
9798
r"""
9899
Return ``True`` if and only if ``x`` is a Baxter permutation of
99100
size ``self._n``.

src/sage/combinat/blob_algebra.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
from sage.combinat.free_module import CombinatorialFreeModule
2626
from sage.combinat.subset import powerset
2727
from sage.misc.cachefunc import cached_method
28-
from sage.rings.integer_ring import ZZ
28+
from sage.rings.integer import Integer
2929
from sage.structure.element import Element, get_coercion_model
3030
from sage.structure.parent import Parent
3131
from sage.structure.richcmp import richcmp
@@ -164,7 +164,7 @@ def __init__(self, n):
164164
sage: BD4 = BlobDiagrams(4)
165165
sage: TestSuite(BD4).run()
166166
"""
167-
self._n = ZZ(n)
167+
self._n = Integer(n)
168168
self._TL_diagrams = TemperleyLiebDiagrams(n)
169169
Parent.__init__(self, category=FiniteEnumeratedSets())
170170

src/sage/combinat/dyck_word.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@
9090
from sage.categories.infinite_enumerated_sets import InfiniteEnumeratedSets
9191
from sage.categories.posets import Posets
9292

93-
from sage.rings.integer_ring import ZZ
93+
from sage.rings.integer import Integer
9494
from sage.rings.rational_field import QQ
9595
from sage.combinat.permutation import Permutation, Permutations
9696
from sage.combinat.words.word import Word
@@ -3286,14 +3286,14 @@ def __classcall_private__(cls, k1=None, k2=None, complete=True):
32863286
return CompleteDyckWords_all()
32873287
return DyckWords_all()
32883288

3289-
k1 = ZZ(k1)
3289+
k1 = Integer(k1)
32903290
if k1 < 0:
32913291
raise ValueError("k1 (= %s) must be nonnegative" % k1)
32923292
return CompleteDyckWords_size(k1)
32933293
else:
3294-
k1 = ZZ(k1)
3294+
k1 = Integer(k1)
32953295

3296-
k2 = ZZ(k2)
3296+
k2 = Integer(k2)
32973297
if k1 < 0 or (k2 is not None and k2 < 0):
32983298
raise ValueError("k1 (= %s) and k2 (= %s) must be nonnegative, with k1 >= k2" % (k1, k2))
32993299
if k1 < k2:
@@ -3638,8 +3638,8 @@ def __init__(self, k1, k2):
36383638
# Dyck paths, not words; having k1 opening parens and k2 closing
36393639
# parens corresponds to paths of length k1 + k2 ending at height
36403640
# k1 - k2.
3641-
k1 = ZZ(k1)
3642-
k2 = ZZ(k2)
3641+
k1 = Integer(k1)
3642+
k2 = Integer(k2)
36433643
self.n = k1 + k2
36443644
self.endht = k1 - k2
36453645

@@ -3697,8 +3697,8 @@ def __init__(self, k1, k2):
36973697
Integer Ring
36983698
sage: TestSuite(DyckWords(4,2)).run()
36993699
"""
3700-
self.k1 = ZZ(k1)
3701-
self.k2 = ZZ(k2)
3700+
self.k1 = Integer(k1)
3701+
self.k2 = Integer(k2)
37023702
DyckWords.__init__(self, category=FiniteEnumeratedSets())
37033703

37043704
def _repr_(self) -> str:

src/sage/combinat/posets/posets.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7167,12 +7167,12 @@ def order_polytope(self):
71677167
True
71687168
"""
71697169
from sage.geometry.polyhedron.constructor import Polyhedron
7170-
ineqs = [[0] + [ZZ(j == v) - ZZ(j == u) for j in self]
7170+
ineqs = [[0] + [Integer(j == v) - Integer(j == u) for j in self]
71717171
for u, v in self.hasse_diagram().edges(sort=False, labels=False)]
71727172
for i in self.maximal_elements():
7173-
ineqs += [[1] + [-ZZ(j == i) for j in self]]
7173+
ineqs += [[1] + [-Integer(j == i) for j in self]]
71747174
for i in self.minimal_elements():
7175-
ineqs += [[0] + [ZZ(j == i) for j in self]]
7175+
ineqs += [[0] + [Integer(j == i) for j in self]]
71767176
return Polyhedron(ieqs=ineqs, base_ring=ZZ)
71777177

71787178
def chain_polytope(self):
@@ -7206,10 +7206,10 @@ def chain_polytope(self):
72067206
A 5-dimensional polyhedron in ZZ^5 defined as the convex hull of 8 vertices
72077207
"""
72087208
from sage.geometry.polyhedron.constructor import Polyhedron
7209-
ineqs = [[1] + [-ZZ(j in chain) for j in self]
7209+
ineqs = [[1] + [-Integer(j in chain) for j in self]
72107210
for chain in self.maximal_chains_iterator()]
72117211
for i in self:
7212-
ineqs += [[0] + [ZZ(j == i) for j in self]]
7212+
ineqs += [[0] + [Integer(j == i) for j in self]]
72137213
return Polyhedron(ieqs=ineqs, base_ring=ZZ)
72147214

72157215
def zeta_polynomial(self):

0 commit comments

Comments
 (0)