CLN: Make Series._values match Index._values

pandas/core/apply.py

@@ -280,6 +280,7 @@ def apply_standard(self):
             and not self.dtypes.apply(is_extension_array_dtype).any()
             # Disallow complex_internals since libreduction shortcut
             #  cannot handle MultiIndex
+            and not self.dtypes.apply(lambda x: x.kind in ["m", "M"]).any()
             and not isinstance(self.agg_axis, ABCMultiIndex)
         ):
 

pandas/core/arrays/numpy_.py

@@ -43,7 +43,6 @@ class PandasDtype(ExtensionDtype):
     def __init__(self, dtype):
         dtype = np.dtype(dtype)
         self._dtype = dtype
-        self._name = dtype.name
         self._type = dtype.type
 
     def __repr__(self) -> str:
@@ -56,7 +55,7 @@ def numpy_dtype(self):
 
     @property
     def name(self):
-        return self._name
+        return self._dtype.name
 
     @property
     def type(self):

pandas/core/base.py

@@ -18,13 +18,11 @@
 from pandas.core.dtypes.cast import is_nested_object
 from pandas.core.dtypes.common import (
     is_categorical_dtype,
-    is_datetime64_ns_dtype,
     is_dict_like,
     is_extension_array_dtype,
     is_list_like,
     is_object_dtype,
     is_scalar,
-    is_timedelta64_ns_dtype,
     needs_i8_conversion,
 )
 from pandas.core.dtypes.generic import ABCDataFrame, ABCIndexClass, ABCSeries
@@ -749,17 +747,7 @@ def array(self) -> ExtensionArray:
         # Special mixin syntax may be developed in the future:
         # https://github.com/python/typing/issues/246
         result = self._values  # type: ignore
-
-        if is_datetime64_ns_dtype(result.dtype):
-            from pandas.arrays import DatetimeArray
-
-            result = DatetimeArray(result)
-        elif is_timedelta64_ns_dtype(result.dtype):
-            from pandas.arrays import TimedeltaArray
-
-            result = TimedeltaArray(result)
-
-        elif not is_extension_array_dtype(result.dtype):
+        if isinstance(result, np.ndarray):
             from pandas.core.arrays.numpy_ import PandasArray
 
             result = PandasArray(result)
@@ -1270,6 +1258,9 @@ def unique(self):
         if hasattr(values, "unique"):
 
             result = values.unique()
+            if self.dtype.kind in ["m", "M"]:
+                if getattr(self.dtype, "tz", None) is None:
+                    result = np.asarray(result)
         else:
             result = unique1d(values)
 

pandas/core/construction.py

@@ -377,7 +377,10 @@ def extract_array(obj, extract_numpy: bool = False):
     array([1, 2, 3])
     """
     if isinstance(obj, (ABCIndexClass, ABCSeries)):
-        obj = obj.array
+        arr = obj._values
+        if not extract_numpy and isinstance(arr, np.ndarray):
+            return obj.array
+        return arr
 
     if extract_numpy and isinstance(obj, ABCPandasArray):
         obj = obj.to_numpy()

pandas/core/internals/blocks.py

@@ -12,6 +12,7 @@
 import pandas._libs.internals as libinternals
 from pandas._libs.tslibs import Timedelta, conversion
 from pandas._libs.tslibs.timezones import tz_compare
+from pandas.util._decorators import cache_readonly
 from pandas.util._validators import validate_bool_kwarg
 
 from pandas.core.dtypes.cast import (
@@ -2112,6 +2113,13 @@ def get_values(self, dtype=None):
             return result.reshape(self.values.shape)
         return self.values
 
+    def internal_values(self):
+        return self._ea_values
+
+    @cache_readonly
+    def _ea_values(self):
+        return self._holder(self.values)
+
 
 class DatetimeBlock(DatetimeLikeBlockMixin, Block):
     __slots__ = ()
@@ -2148,6 +2156,7 @@ def _maybe_coerce_values(self, values):
             values = values._data
 
         assert isinstance(values, np.ndarray), type(values)
+        assert values.dtype == _NS_DTYPE, values.dtype
         return values
 
     def astype(self, dtype, copy: bool = False, errors: str = "raise"):
@@ -2242,6 +2251,7 @@ class DatetimeTZBlock(ExtensionBlock, DatetimeBlock):
     is_datetimetz = True
     is_extension = True
 
+    internal_values = Block.internal_values
     _can_hold_element = DatetimeBlock._can_hold_element
     to_native_types = DatetimeBlock.to_native_types
     fill_value = np.datetime64("NaT", "ns")

pandas/tests/indexes/datetimes/test_tools.py

@@ -1583,7 +1583,7 @@ def test_string_na_nat_conversion(self, cache):
         for i in range(5):
             x = series[i]
             if isna(x):
-                expected[i] = iNaT
+                expected[i] = pd.NaT
             else:
                 expected[i] = to_datetime(x, cache=cache)
 

pandas/tests/reductions/test_reductions.py

@@ -316,7 +316,12 @@ def test_invalid_td64_reductions(self, opname):
         )
         td = s.diff()
 
-        msg = "reduction operation '{op}' not allowed for this dtype"
+        msg = "|".join(
+            [
+                "reduction operation '{op}' not allowed for this dtype",
+                r"cannot perform {op} with type timedelta64\[ns\]",
+            ]
+        )
         msg = msg.format(op=opname)
 
         with pytest.raises(TypeError, match=msg):
@@ -648,7 +653,13 @@ def test_ops_consistency_on_empty(self, method):
         # timedelta64[ns]
         tdser = Series([], dtype="m8[ns]")
         if method == "var":
-            with pytest.raises(TypeError, match="operation 'var' not allowed"):
+            msg = "|".join(
+                [
+                    "operation 'var' not allowed",
+                    r"cannot perform var with type timedelta64\[ns\]",
+                ]
+            )
+            with pytest.raises(TypeError, match=msg):
                 getattr(tdser, method)()
         else:
             result = getattr(tdser, method)()