Skip to content

Commit 782b542

Browse files
jbrockmendelTLouf
authored andcommitted
CLN: remove unused _max_fitting_element (pandas-dev#41639)
1 parent c2bca61 commit 782b542

File tree

2 files changed

+0
-23
lines changed

2 files changed

+0
-23
lines changed

pandas/core/indexes/range.py

-5
Original file line numberDiff line numberDiff line change
@@ -604,11 +604,6 @@ def _min_fitting_element(self, lower_limit: int) -> int:
604604
no_steps = -(-(lower_limit - self.start) // abs(self.step))
605605
return self.start + abs(self.step) * no_steps
606606

607-
def _max_fitting_element(self, upper_limit: int) -> int:
608-
"""Returns the largest element smaller than or equal to the limit"""
609-
no_steps = (upper_limit - self.start) // abs(self.step)
610-
return self.start + abs(self.step) * no_steps
611-
612607
def _extended_gcd(self, a: int, b: int) -> tuple[int, int, int]:
613608
"""
614609
Extended Euclidean algorithms to solve Bezout's identity:

pandas/tests/indexes/ranges/test_range.py

-18
Original file line numberDiff line numberDiff line change
@@ -369,24 +369,6 @@ def test_min_fitting_element(self):
369369
result = RangeIndex(5, big_num * 2, 1)._min_fitting_element(big_num)
370370
assert big_num == result
371371

372-
def test_max_fitting_element(self):
373-
result = RangeIndex(0, 20, 2)._max_fitting_element(17)
374-
assert 16 == result
375-
376-
result = RangeIndex(1, 6)._max_fitting_element(4)
377-
assert 4 == result
378-
379-
result = RangeIndex(18, -2, -2)._max_fitting_element(17)
380-
assert 16 == result
381-
382-
result = RangeIndex(5, 0, -1)._max_fitting_element(4)
383-
assert 4 == result
384-
385-
big_num = 500000000000000000000000
386-
387-
result = RangeIndex(5, big_num * 2, 1)._max_fitting_element(big_num)
388-
assert big_num == result
389-
390372
def test_pickle_compat_construction(self):
391373
# RangeIndex() is a valid constructor
392374
pass

0 commit comments

Comments
 (0)