Revert "TYP: Improve typing interval inclusive (pandas-dev#47646)"

This reverts commit 5506476.

Author: mroeschke

pandas/_libs/interval.pyi

@@ -10,7 +10,7 @@ import numpy.typing as npt
 
 from pandas._libs import lib
 from pandas._typing import (
-    IntervalInclusiveType,
+    IntervalClosedType,
     Timedelta,
     Timestamp,
 )
@@ -54,25 +54,25 @@ class IntervalMixin:
 
 def _warning_interval(
     inclusive, closed
-) -> tuple[IntervalInclusiveType, lib.NoDefault]: ...
+) -> tuple[IntervalClosedType, lib.NoDefault]: ...
 
 class Interval(IntervalMixin, Generic[_OrderableT]):
     @property
     def left(self: Interval[_OrderableT]) -> _OrderableT: ...
     @property
     def right(self: Interval[_OrderableT]) -> _OrderableT: ...
     @property
-    def inclusive(self) -> IntervalInclusiveType: ...
+    def inclusive(self) -> IntervalClosedType: ...
     @property
-    def closed(self) -> IntervalInclusiveType: ...
+    def closed(self) -> IntervalClosedType: ...
     mid: _MidDescriptor
     length: _LengthDescriptor
     def __init__(
         self,
         left: _OrderableT,
         right: _OrderableT,
-        inclusive: IntervalInclusiveType = ...,
-        closed: IntervalInclusiveType = ...,
+        inclusive: IntervalClosedType = ...,
+        closed: IntervalClosedType = ...,
     ) -> None: ...
     def __hash__(self) -> int: ...
     @overload
@@ -156,14 +156,14 @@ class Interval(IntervalMixin, Generic[_OrderableT]):
 
 def intervals_to_interval_bounds(
     intervals: np.ndarray, validate_closed: bool = ...
-) -> tuple[np.ndarray, np.ndarray, IntervalInclusiveType]: ...
+) -> tuple[np.ndarray, np.ndarray, str]: ...
 
 class IntervalTree(IntervalMixin):
     def __init__(
         self,
         left: np.ndarray,
         right: np.ndarray,
-        inclusive: IntervalInclusiveType = ...,
+        inclusive: IntervalClosedType = ...,
         leaf_size: int = ...,
     ) -> None: ...
     @property

pandas/_typing.py

@@ -313,7 +313,7 @@ def closed(self) -> bool:
 
 # Interval closed type
 IntervalLeftRight = Literal["left", "right"]
-IntervalInclusiveType = Union[IntervalLeftRight, Literal["both", "neither"]]
+IntervalClosedType = Union[IntervalLeftRight, Literal["both", "neither"]]
 
 # datetime and NaTType
 DatetimeNaTType = Union[datetime, "NaTType"]

pandas/core/arrays/interval.py

@@ -33,7 +33,7 @@
 from pandas._typing import (
     ArrayLike,
     Dtype,
-    IntervalInclusiveType,
+    IntervalClosedType,
     NpDtype,
     PositionalIndexer,
     ScalarIndexer,
@@ -231,7 +231,7 @@ def ndim(self) -> Literal[1]:
     def __new__(
         cls: type[IntervalArrayT],
         data,
-        inclusive: IntervalInclusiveType | None = None,
+        inclusive: str | None = None,
         dtype: Dtype | None = None,
         copy: bool = False,
         verify_integrity: bool = True,
@@ -278,7 +278,7 @@ def _simple_new(
         cls: type[IntervalArrayT],
         left,
         right,
-        inclusive: IntervalInclusiveType | None = None,
+        inclusive=None,
         copy: bool = False,
         dtype: Dtype | None = None,
         verify_integrity: bool = True,
@@ -432,7 +432,7 @@ def _from_factorized(
     def from_breaks(
         cls: type[IntervalArrayT],
         breaks,
-        inclusive: IntervalInclusiveType | None = None,
+        inclusive: IntervalClosedType | None = None,
         copy: bool = False,
         dtype: Dtype | None = None,
     ) -> IntervalArrayT:
@@ -514,7 +514,7 @@ def from_arrays(
         cls: type[IntervalArrayT],
         left,
         right,
-        inclusive: IntervalInclusiveType | None = None,
+        inclusive: IntervalClosedType | None = None,
         copy: bool = False,
         dtype: Dtype | None = None,
     ) -> IntervalArrayT:
@@ -587,7 +587,7 @@ def from_arrays(
     def from_tuples(
         cls: type[IntervalArrayT],
         data,
-        inclusive: IntervalInclusiveType | None = None,
+        inclusive=None,
         copy: bool = False,
         dtype: Dtype | None = None,
     ) -> IntervalArrayT:
@@ -1362,15 +1362,15 @@ def overlaps(self, other):
     # ---------------------------------------------------------------------
 
     @property
-    def inclusive(self) -> IntervalInclusiveType:
+    def inclusive(self) -> IntervalClosedType:
         """
         Whether the intervals are closed on the left-side, right-side, both or
         neither.
         """
         return self.dtype.inclusive
 
     @property
-    def closed(self) -> IntervalInclusiveType:
+    def closed(self) -> IntervalClosedType:
         """
         String describing the inclusive side the intervals.
 
@@ -1424,9 +1424,7 @@ def closed(self) -> IntervalInclusiveType:
             ),
         }
     )
-    def set_closed(
-        self: IntervalArrayT, closed: IntervalInclusiveType
-    ) -> IntervalArrayT:
+    def set_closed(self: IntervalArrayT, closed: IntervalClosedType) -> IntervalArrayT:
         warnings.warn(
             "set_closed is deprecated and will be removed in a future version. "
             "Use set_inclusive instead.",
@@ -1477,7 +1475,7 @@ def set_closed(
         }
     )
     def set_inclusive(
-        self: IntervalArrayT, inclusive: IntervalInclusiveType
+        self: IntervalArrayT, inclusive: IntervalClosedType
     ) -> IntervalArrayT:
         if inclusive not in VALID_CLOSED:
             msg = f"invalid option for 'inclusive': {inclusive}"

pandas/core/dtypes/dtypes.py

@@ -39,7 +39,6 @@
 from pandas._typing import (
     Dtype,
     DtypeObj,
-    IntervalInclusiveType,
     Ordered,
     npt,
     type_t,
@@ -1096,7 +1095,7 @@ class IntervalDtype(PandasExtensionDtype):
     def __new__(
         cls,
         subtype=None,
-        inclusive: IntervalInclusiveType | None = None,
+        inclusive: str_type | None = None,
         closed: None | lib.NoDefault = lib.no_default,
     ):
         from pandas.core.dtypes.common import (
@@ -1145,11 +1144,7 @@ def __new__(
                                     "'inclusive' keyword does not match value "
                                     "specified in dtype string"
                                 )
-                        # Incompatible types in assignment (expression has type
-                        # "Union[str, Any]", variable has type
-                        # "Optional[Union[Literal['left', 'right'],
-                        # Literal['both', 'neither']]]")
-                        inclusive = gd["inclusive"]  # type: ignore[assignment]
+                        inclusive = gd["inclusive"]
 
             try:
                 subtype = pandas_dtype(subtype)

pandas/core/generic.py

@@ -56,7 +56,7 @@
     IgnoreRaise,
     IndexKeyFunc,
     IndexLabel,
-    IntervalInclusiveType,
+    IntervalClosedType,
     JSONSerializable,
     Level,
     Manager,
@@ -8221,7 +8221,7 @@ def between_time(
         end_time,
         include_start: bool_t | lib.NoDefault = lib.no_default,
         include_end: bool_t | lib.NoDefault = lib.no_default,
-        inclusive: IntervalInclusiveType | None = None,
+        inclusive: IntervalClosedType | None = None,
         axis=None,
     ) -> NDFrameT:
         """
@@ -8327,7 +8327,7 @@ def between_time(
             left = True if include_start is lib.no_default else include_start
             right = True if include_end is lib.no_default else include_end
 
-            inc_dict: dict[tuple[bool_t, bool_t], IntervalInclusiveType] = {
+            inc_dict: dict[tuple[bool_t, bool_t], IntervalClosedType] = {
                 (True, True): "both",
                 (True, False): "left",
                 (False, True): "right",

pandas/core/indexes/datetimes.py

@@ -37,7 +37,7 @@
 from pandas._typing import (
     Dtype,
     DtypeObj,
-    IntervalInclusiveType,
+    IntervalClosedType,
     IntervalLeftRight,
     npt,
 )
@@ -926,7 +926,7 @@ def date_range(
     normalize: bool = False,
     name: Hashable = None,
     closed: Literal["left", "right"] | None | lib.NoDefault = lib.no_default,
-    inclusive: IntervalInclusiveType | None = None,
+    inclusive: IntervalClosedType | None = None,
     **kwargs,
 ) -> DatetimeIndex:
     """
@@ -1132,7 +1132,7 @@ def bdate_range(
     weekmask=None,
     holidays=None,
     closed: IntervalLeftRight | lib.NoDefault | None = lib.no_default,
-    inclusive: IntervalInclusiveType | None = None,
+    inclusive: IntervalClosedType | None = None,
     **kwargs,
 ) -> DatetimeIndex:
     """

pandas/core/indexes/interval.py

@@ -31,7 +31,7 @@
 from pandas._typing import (
     Dtype,
     DtypeObj,
-    IntervalInclusiveType,
+    IntervalClosedType,
     npt,
 )
 from pandas.errors import InvalidIndexError
@@ -199,7 +199,7 @@ class IntervalIndex(ExtensionIndex):
     _typ = "intervalindex"
 
     # annotate properties pinned via inherit_names
-    inclusive: IntervalInclusiveType
+    inclusive: IntervalClosedType
     is_non_overlapping_monotonic: bool
     closed_left: bool
     closed_right: bool
@@ -218,7 +218,7 @@ class IntervalIndex(ExtensionIndex):
     def __new__(
         cls,
         data,
-        inclusive: IntervalInclusiveType | None = None,
+        inclusive=None,
         dtype: Dtype | None = None,
         copy: bool = False,
         name: Hashable = None,
@@ -267,7 +267,7 @@ def closed(self):
     def from_breaks(
         cls,
         breaks,
-        inclusive: IntervalInclusiveType | None = None,
+        inclusive=None,
         name: Hashable = None,
         copy: bool = False,
         dtype: Dtype | None = None,
@@ -303,7 +303,7 @@ def from_arrays(
         cls,
         left,
         right,
-        inclusive: IntervalInclusiveType | None = None,
+        inclusive=None,
         name: Hashable = None,
         copy: bool = False,
         dtype: Dtype | None = None,
@@ -338,7 +338,7 @@ def from_arrays(
     def from_tuples(
         cls,
         data,
-        inclusive: IntervalInclusiveType | None = None,
+        inclusive=None,
         name: Hashable = None,
         copy: bool = False,
         dtype: Dtype | None = None,
@@ -990,7 +990,7 @@ def interval_range(
     periods=None,
     freq=None,
     name: Hashable = None,
-    inclusive: IntervalInclusiveType | None = None,
+    inclusive: IntervalClosedType | None = None,
 ) -> IntervalIndex:
     """
     Return a fixed frequency IntervalIndex.

pandas/io/formats/style.py

@@ -26,7 +26,6 @@
     Axis,
     FilePath,
     IndexLabel,
-    IntervalInclusiveType,
     Level,
     QuantileInterpolation,
     Scalar,
@@ -3489,7 +3488,7 @@ def highlight_between(
         axis: Axis | None = 0,
         left: Scalar | Sequence | None = None,
         right: Scalar | Sequence | None = None,
-        inclusive: IntervalInclusiveType = "both",
+        inclusive: str = "both",
         props: str | None = None,
     ) -> Styler:
         """
@@ -3594,7 +3593,7 @@ def highlight_quantile(
         q_left: float = 0.0,
         q_right: float = 1.0,
         interpolation: QuantileInterpolation = "linear",
-        inclusive: IntervalInclusiveType = "both",
+        inclusive: str = "both",
         props: str | None = None,
     ) -> Styler:
         """
@@ -3979,7 +3978,7 @@ def _highlight_between(
     props: str,
     left: Scalar | Sequence | np.ndarray | NDFrame | None = None,
     right: Scalar | Sequence | np.ndarray | NDFrame | None = None,
-    inclusive: bool | IntervalInclusiveType = True,
+    inclusive: bool | str = True,
 ) -> np.ndarray:
     """
     Return an array of css props based on condition of data values within given range.

pandas/util/_validators.py

@@ -16,7 +16,6 @@
 
 import numpy as np
 
-from pandas._typing import IntervalInclusiveType
 from pandas.util._exceptions import find_stack_level
 
 from pandas.core.dtypes.common import (
@@ -494,7 +493,7 @@ def validate_endpoints(closed: str | None) -> tuple[bool, bool]:
     return left_closed, right_closed
 
 
-def validate_inclusive(inclusive: IntervalInclusiveType | None) -> tuple[bool, bool]:
+def validate_inclusive(inclusive: str | None) -> tuple[bool, bool]:
     """
     Check that the `inclusive` argument is among {"both", "neither", "left", "right"}.