Skip to content

Commit 64a5b9b

Browse files
committed
CLN: make some privte Index methods keyword-only
1 parent 5675cd8 commit 64a5b9b

File tree

4 files changed

+11
-8
lines changed

4 files changed

+11
-8
lines changed

pandas/core/generic.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1152,7 +1152,7 @@ def rename(
11521152
]
11531153
raise KeyError(f"{missing_labels} not found in axis")
11541154

1155-
new_index = ax._transform_index(f, level)
1155+
new_index = ax._transform_index(f, level=level)
11561156
result._set_axis_nocheck(new_index, axis=axis_no, inplace=True)
11571157
result._clear_item_cache()
11581158

pandas/core/groupby/grouper.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -490,7 +490,7 @@ def __init__(
490490
self.grouping_vector, # Index
491491
self._codes,
492492
self._group_index,
493-
) = index._get_grouper_for_level(mapper, ilevel)
493+
) = index._get_grouper_for_level(mapper, level=ilevel)
494494

495495
# a passed Grouper like, directly get the grouper in the same way
496496
# as single grouper groupby, use the group_info to get codes

pandas/core/indexes/base.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1500,7 +1500,7 @@ def _validate_names(
15001500
def _get_names(self) -> FrozenList:
15011501
return FrozenList((self.name,))
15021502

1503-
def _set_names(self, values, level=None) -> None:
1503+
def _set_names(self, values, *, level=None) -> None:
15041504
"""
15051505
Set new names on index. Each name has to be a hashable type.
15061506
@@ -1906,7 +1906,7 @@ def _drop_level_numbers(self, levnums: list[int]):
19061906
verify_integrity=False,
19071907
)
19081908

1909-
def _get_grouper_for_level(self, mapper, level=None):
1909+
def _get_grouper_for_level(self, mapper, *, level=None):
19101910
"""
19111911
Get index grouper corresponding to an index level
19121912
@@ -1915,7 +1915,7 @@ def _get_grouper_for_level(self, mapper, level=None):
19151915
mapper: Group mapping function or None
19161916
Function mapping index values to groups
19171917
level : int or None
1918-
Index level
1918+
Index level, positional
19191919
19201920
Returns
19211921
-------
@@ -5642,7 +5642,7 @@ def map(self, mapper, na_action=None):
56425642

56435643
# TODO: De-duplicate with map, xref GH#32349
56445644
@final
5645-
def _transform_index(self, func, level=None) -> Index:
5645+
def _transform_index(self, func, *, level=None) -> Index:
56465646
"""
56475647
Apply function to all values found in index.
56485648
@@ -5651,6 +5651,7 @@ def _transform_index(self, func, level=None) -> Index:
56515651
"""
56525652
if isinstance(self, ABCMultiIndex):
56535653
if level is not None:
5654+
# Caller is responsible for ensuring level is positional.
56545655
items = [
56555656
tuple(func(y) if i == level else y for i, y in enumerate(x))
56565657
for x in self

pandas/core/indexes/multi.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -761,6 +761,7 @@ def levels(self) -> FrozenList:
761761
def _set_levels(
762762
self,
763763
levels,
764+
*,
764765
level=None,
765766
copy: bool = False,
766767
validate: bool = True,
@@ -954,6 +955,7 @@ def codes(self):
954955
def _set_codes(
955956
self,
956957
codes,
958+
*,
957959
level=None,
958960
copy: bool = False,
959961
validate: bool = True,
@@ -1392,7 +1394,7 @@ def format(
13921394
def _get_names(self) -> FrozenList:
13931395
return FrozenList(self._names)
13941396

1395-
def _set_names(self, names, level=None, validate: bool = True):
1397+
def _set_names(self, names, *, level=None, validate: bool = True):
13961398
"""
13971399
Set new names on index. Each name has to be a hashable type.
13981400
@@ -1473,7 +1475,7 @@ def _set_names(self, names, level=None, validate: bool = True):
14731475
# --------------------------------------------------------------------
14741476

14751477
@doc(Index._get_grouper_for_level)
1476-
def _get_grouper_for_level(self, mapper, level):
1478+
def _get_grouper_for_level(self, mapper, *, level):
14771479
indexer = self.codes[level]
14781480
level_index = self.levels[level]
14791481

0 commit comments

Comments
 (0)