CLN: remove redundant _get_unique_index (pandas-dev#42184)

Author: jbrockmendel

pandas/core/indexes/base.py

@@ -2754,16 +2754,6 @@ def duplicated(self, keep: Literal["first", "last", False] = "first") -> np.ndar
             return np.zeros(len(self), dtype=bool)
         return self._duplicated(keep=keep)
 
-    def _get_unique_index(self: _IndexT) -> _IndexT:
-        """
-        Returns an index containing unique values.
-
-        Returns
-        -------
-        Index
-        """
-        return self.unique()
-
     # --------------------------------------------------------------------
     # Arithmetic & Logical Methods
 
@@ -3200,7 +3190,7 @@ def difference(self, other, sort=None):
 
     def _difference(self, other, sort):
 
-        this = self._get_unique_index()
+        this = self.unique()
 
         indexer = this.get_indexer_for(other)
         indexer = indexer.take((indexer != -1).nonzero()[0])

pandas/core/indexes/extension.py

@@ -364,13 +364,6 @@ def _validate_fill_value(self, value):
         """
         return self._data._validate_setitem_value(value)
 
-    def _get_unique_index(self):
-        if self.is_unique:
-            return self
-
-        result = self._data.unique()
-        return type(self)._simple_new(result, name=self.name)
-
     @doc(Index.map)
     def map(self, mapper, na_action=None):
         # Try to run function on index first, and then on elements of index

pandas/tests/indexes/multi/test_duplicates.py

@@ -74,15 +74,6 @@ def test_unique_level(idx, level):
     tm.assert_index_equal(result, expected)
 
 
-def test_get_unique_index(idx):
-    mi = idx[[0, 1, 0, 1, 1, 0, 0]]
-    expected = mi._shallow_copy(mi[[0, 1]])
-
-    result = mi._get_unique_index()
-    assert result.unique
-    tm.assert_index_equal(result, expected)
-
-
 def test_duplicate_multiindex_codes():
     # GH 17464
     # Make sure that a MultiIndex with duplicate levels throws a ValueError

pandas/tests/indexes/test_common.py

@@ -126,7 +126,7 @@ def test_copy_and_deepcopy(self, index_flat):
         new_copy = index.copy(deep=True, name="banana")
         assert new_copy.name == "banana"
 
-    def test_unique(self, index_flat):
+    def test_unique_level(self, index_flat):
         # don't test a MultiIndex here (as its tested separated)
         index = index_flat
 
@@ -147,7 +147,7 @@ def test_unique(self, index_flat):
         with pytest.raises(KeyError, match=msg):
             index.unique(level="wrong")
 
-    def test_get_unique_index(self, index_flat):
+    def test_unique(self, index_flat):
         # MultiIndex tested separately
         index = index_flat
         if not len(index):
@@ -164,7 +164,7 @@ def test_get_unique_index(self, index_flat):
         except NotImplementedError:
             pass
 
-        result = idx._get_unique_index()
+        result = idx.unique()
         tm.assert_index_equal(result, idx_unique)
 
         # nans:
@@ -195,7 +195,7 @@ def test_get_unique_index(self, index_flat):
 
         expected = idx_unique_nan
         for i in [idx_nan, idx_unique_nan]:
-            result = i._get_unique_index()
+            result = i.unique()
             tm.assert_index_equal(result, expected)
 
     def test_searchsorted_monotonic(self, index_flat):