Skip to content

Commit 4fa7e00

Browse files
committed
#157 Rename type hint (DataType → AnyXarray)
1 parent 9f350c0 commit 4fa7e00

File tree

5 files changed

+18
-19
lines changed

5 files changed

+18
-19
lines changed

xarray_dataclasses/dataarray.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@
2828
# submodules
2929
from .datamodel import DataModel
3030
from .dataoptions import DataOptions
31-
from .typing import AnyArray, AnyField, DataClass, DataType, Order, Shape, Sizes
31+
from .typing import AnyArray, AnyField, AnyXarray, DataClass, Order, Shape, Sizes
3232

3333

3434
# type hints
@@ -50,7 +50,7 @@ def __init__(self, *args: PInit.args, **kwargs: PInit.kwargs) -> None:
5050
@overload
5151
def asdataarray(
5252
dataclass: OptionedClass[PInit, TDataArray],
53-
reference: Optional[DataType] = None,
53+
reference: Optional[AnyXarray] = None,
5454
dataoptions: None = None,
5555
) -> TDataArray:
5656
...
@@ -59,7 +59,7 @@ def asdataarray(
5959
@overload
6060
def asdataarray(
6161
dataclass: DataClass[PInit],
62-
reference: Optional[DataType] = None,
62+
reference: Optional[AnyXarray] = None,
6363
dataoptions: None = None,
6464
) -> xr.DataArray:
6565
...
@@ -68,15 +68,15 @@ def asdataarray(
6868
@overload
6969
def asdataarray(
7070
dataclass: Any,
71-
reference: Optional[DataType] = None,
71+
reference: Optional[AnyXarray] = None,
7272
dataoptions: DataOptions[TDataArray] = DataOptions(xr.DataArray),
7373
) -> TDataArray:
7474
...
7575

7676

7777
def asdataarray(
7878
dataclass: Any,
79-
reference: Optional[DataType] = None,
79+
reference: Optional[AnyXarray] = None,
8080
dataoptions: Any = None,
8181
) -> Any:
8282
"""Create a DataArray object from a dataclass object.

xarray_dataclasses/datamodel.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
AnyDType,
1919
AnyField,
2020
DataClass,
21-
DataType,
21+
AnyXarray,
2222
Dims,
2323
FType,
2424
get_annotated,
@@ -120,7 +120,7 @@ def __post_init__(self) -> None:
120120
if model.names:
121121
setattr(self, "name", model.names[0].value)
122122

123-
def __call__(self, reference: Optional[DataType] = None) -> xr.DataArray:
123+
def __call__(self, reference: Optional[AnyXarray] = None) -> xr.DataArray:
124124
"""Create a DataArray object according to the entry."""
125125
from .dataarray import asdataarray
126126

@@ -240,7 +240,7 @@ def get_typedarray(
240240
data: Any,
241241
dims: Dims,
242242
dtype: Optional[AnyDType],
243-
reference: Optional[DataType] = None,
243+
reference: Optional[AnyXarray] = None,
244244
) -> xr.DataArray:
245245
"""Create a DataArray object with given dims and dtype.
246246

xarray_dataclasses/dataoptions.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,17 +8,17 @@
88

99

1010
# submodules
11-
from .typing import DataType
11+
from .typing import AnyXarray
1212

1313

1414
# type hints
15-
TDataType = TypeVar("TDataType", bound=DataType)
15+
TAnyXarray = TypeVar("TAnyXarray", bound=AnyXarray)
1616

1717

1818
# dataclasses
1919
@dataclass(frozen=True)
20-
class DataOptions(Generic[TDataType]):
20+
class DataOptions(Generic[TAnyXarray]):
2121
"""Options for DataArray or Dataset creation."""
2222

23-
factory: Callable[..., TDataType]
23+
factory: Callable[..., TAnyXarray]
2424
"""Factory function for DataArray or Dataset."""

xarray_dataclasses/dataset.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
# submodules
1919
from .datamodel import DataModel
2020
from .dataoptions import DataOptions
21-
from .typing import AnyArray, AnyField, DataClass, DataType, Order, Shape, Sizes
21+
from .typing import AnyArray, AnyField, AnyXarray, DataClass, Order, Shape, Sizes
2222

2323

2424
# type hints
@@ -40,7 +40,7 @@ def __init__(self, *args: PInit.args, **kwargs: PInit.kwargs) -> None:
4040
@overload
4141
def asdataset(
4242
dataclass: OptionedClass[PInit, TDataset],
43-
reference: Optional[DataType] = None,
43+
reference: Optional[AnyXarray] = None,
4444
dataoptions: None = None,
4545
) -> TDataset:
4646
...
@@ -49,7 +49,7 @@ def asdataset(
4949
@overload
5050
def asdataset(
5151
dataclass: DataClass[PInit],
52-
reference: Optional[DataType] = None,
52+
reference: Optional[AnyXarray] = None,
5353
dataoptions: None = None,
5454
) -> xr.Dataset:
5555
...
@@ -58,15 +58,15 @@ def asdataset(
5858
@overload
5959
def asdataset(
6060
dataclass: Any,
61-
reference: Optional[DataType] = None,
61+
reference: Optional[AnyXarray] = None,
6262
dataoptions: DataOptions[TDataset] = DataOptions(xr.Dataset),
6363
) -> TDataset:
6464
...
6565

6666

6767
def asdataset(
6868
dataclass: Any,
69-
reference: Optional[DataType] = None,
69+
reference: Optional[AnyXarray] = None,
7070
dataoptions: Any = None,
7171
) -> Any:
7272
"""Create a Dataset object from a dataclass object.

xarray_dataclasses/typing.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,9 +64,8 @@
6464
AnyArray: TypeAlias = "np.ndarray[Any, Any]"
6565
AnyDType: TypeAlias = "np.dtype[Any]"
6666
AnyField: TypeAlias = "Field[Any]"
67-
DataType = Union[xr.DataArray, xr.Dataset]
67+
AnyXarray = Union[xr.DataArray, xr.Dataset]
6868
Dims = Tuple[str, ...]
69-
Dtype = Optional[str]
7069
Order = Literal["C", "F"]
7170
Shape = Union[Sequence[int], int]
7271
Sizes = Dict[str, int]

0 commit comments

Comments
 (0)