Skip to content

Commit 39e3412

Browse files
authored
CLN: avoid using internals methods for DataFrame.drop_duplicates (#33075)
1 parent ba72b59 commit 39e3412

File tree

1 file changed

+6
-12
lines changed

1 file changed

+6
-12
lines changed

pandas/core/frame.py

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -4678,22 +4678,16 @@ def drop_duplicates(
46784678
inplace = validate_bool_kwarg(inplace, "inplace")
46794679
duplicated = self.duplicated(subset, keep=keep)
46804680

4681-
if inplace:
4682-
(inds,) = np.asarray(-duplicated).nonzero()
4683-
new_data = self._data.take(inds)
4681+
result = self[-duplicated]
4682+
if ignore_index:
4683+
result.index = ibase.default_index(len(result))
46844684

4685-
if ignore_index:
4686-
new_data.axes[1] = ibase.default_index(len(inds))
4687-
self._update_inplace(new_data)
4685+
if inplace:
4686+
self._update_inplace(result._data)
4687+
return None
46884688
else:
4689-
result = self[-duplicated]
4690-
4691-
if ignore_index:
4692-
result.index = ibase.default_index(len(result))
46934689
return result
46944690

4695-
return None
4696-
46974691
def duplicated(
46984692
self,
46994693
subset: Optional[Union[Hashable, Sequence[Hashable]]] = None,

0 commit comments

Comments
 (0)