Skip to content

Commit b30d7d4

Browse files
authored
CLN: remove kwargs from signature of (Index|MultiIndex).copy (#31669)
1 parent a76cfbf commit b30d7d4

File tree

2 files changed

+11
-10
lines changed

2 files changed

+11
-10
lines changed

pandas/core/indexes/base.py

+6-5
Original file line numberDiff line numberDiff line change
@@ -823,20 +823,22 @@ def repeat(self, repeats, axis=None):
823823
# --------------------------------------------------------------------
824824
# Copying Methods
825825

826-
def copy(self, name=None, deep=False, dtype=None, **kwargs):
826+
def copy(self, name=None, deep=False, dtype=None, names=None):
827827
"""
828828
Make a copy of this object. Name and dtype sets those attributes on
829829
the new object.
830830
831831
Parameters
832832
----------
833-
name : str, optional
833+
name : Label
834834
deep : bool, default False
835-
dtype : numpy dtype or pandas type
835+
dtype : numpy dtype or pandas type, optional
836+
names : list-like, optional
837+
Kept for compatibility with MultiIndex. Should not be used.
836838
837839
Returns
838840
-------
839-
copy : Index
841+
Index
840842
841843
Notes
842844
-----
@@ -848,7 +850,6 @@ def copy(self, name=None, deep=False, dtype=None, **kwargs):
848850
else:
849851
new_index = self._shallow_copy()
850852

851-
names = kwargs.get("names")
852853
names = self._validate_names(name=name, names=names, deep=deep)
853854
new_index = new_index.set_names(names)
854855

pandas/core/indexes/multi.py

+5-5
Original file line numberDiff line numberDiff line change
@@ -1013,8 +1013,8 @@ def copy(
10131013
levels=None,
10141014
codes=None,
10151015
deep=False,
1016+
name=None,
10161017
_set_identity=False,
1017-
**kwargs,
10181018
):
10191019
"""
10201020
Make a copy of this object. Names, dtype, levels and codes can be
@@ -1026,21 +1026,21 @@ def copy(
10261026
dtype : numpy dtype or pandas type, optional
10271027
levels : sequence, optional
10281028
codes : sequence, optional
1029+
deep : bool, default False
1030+
name : Label
1031+
Kept for compatibility with 1-dimensional Index. Should not be used.
10291032
10301033
Returns
10311034
-------
1032-
copy : MultiIndex
1035+
MultiIndex
10331036
10341037
Notes
10351038
-----
10361039
In most cases, there should be no functional difference from using
10371040
``deep``, but if ``deep`` is passed it will attempt to deepcopy.
10381041
This could be potentially expensive on large MultiIndex objects.
10391042
"""
1040-
name = kwargs.get("name")
10411043
names = self._validate_names(name=name, names=names, deep=deep)
1042-
if "labels" in kwargs:
1043-
raise TypeError("'labels' argument has been removed; use 'codes' instead")
10441044
if deep:
10451045
from copy import deepcopy
10461046

0 commit comments

Comments
 (0)