Skip to content

Commit 789f9ec

Browse files
committed
#91 Rename class attribute (key → name)
1 parent 52f714c commit 789f9ec

File tree

4 files changed

+24
-24
lines changed

4 files changed

+24
-24
lines changed

tests/test_datamodel.py

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -54,28 +54,28 @@ class ColorImage:
5454
# test functions
5555
def test_xaxis_attr() -> None:
5656
item = next(iter(xaxis_model.attr.values()))
57-
assert item.key == "units"
57+
assert item.name == "units"
5858
assert item.value == "pixel"
5959
assert item.type == "builtins.str"
6060

6161

6262
def test_xaxis_data() -> None:
6363
item = next(iter(xaxis_model.data.values()))
64-
assert item.key == "data"
64+
assert item.name == "data"
6565
assert item.type == {"dims": ("x",), "dtype": "int"}
6666
assert item.factory is None
6767

6868

6969
def test_yaxis_attr() -> None:
7070
item = next(iter(yaxis_model.attr.values()))
71-
assert item.key == "units"
71+
assert item.name == "units"
7272
assert item.value == "pixel"
7373
assert item.type == "builtins.str"
7474

7575

7676
def test_yaxis_data() -> None:
7777
item = next(iter(yaxis_model.data.values()))
78-
assert item.key == "data"
78+
assert item.name == "data"
7979
assert item.type == {"dims": ("y",), "dtype": "int"}
8080
assert item.factory is None
8181

@@ -84,24 +84,24 @@ def test_image_coord() -> None:
8484
items = iter(image_model.coord.values())
8585

8686
item = next(items)
87-
assert item.key == "mask"
87+
assert item.name == "mask"
8888
assert item.type == {"dims": ("x", "y"), "dtype": "bool"}
8989
assert item.factory is None
9090

9191
item = next(items)
92-
assert item.key == "x"
92+
assert item.name == "x"
9393
assert item.type == {"dims": ("x",), "dtype": "int"}
9494
assert item.factory is XAxis
9595

9696
item = next(items)
97-
assert item.key == "y"
97+
assert item.name == "y"
9898
assert item.type == {"dims": ("y",), "dtype": "int"}
9999
assert item.factory is YAxis
100100

101101

102102
def test_image_data() -> None:
103103
item = next(iter(image_model.data.values()))
104-
assert item.key == "data"
104+
assert item.name == "data"
105105
assert item.type == {"dims": ("x", "y"), "dtype": "float"}
106106
assert item.factory is None
107107

@@ -110,16 +110,16 @@ def test_color_data() -> None:
110110
items = iter(color_model.data.values())
111111

112112
item = next(items)
113-
assert item.key == "red"
113+
assert item.name == "red"
114114
assert item.type == {"dims": ("x", "y"), "dtype": "float"}
115115
assert item.factory is Image
116116

117117
item = next(items)
118-
assert item.key == "green"
118+
assert item.name == "green"
119119
assert item.type == {"dims": ("x", "y"), "dtype": "float"}
120120
assert item.factory is Image
121121

122122
item = next(items)
123-
assert item.key == "blue"
123+
assert item.name == "blue"
124124
assert item.type == {"dims": ("x", "y"), "dtype": "float"}
125125
assert item.factory is Image

xarray_dataclasses/dataarray.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -117,15 +117,15 @@ def asdataarray(
117117
dataarray = dataoptions.factory(item(reference))
118118

119119
for item in model.coord.values():
120-
if item.key in dataarray.dims:
121-
dataarray.coords[item.key] = item(dataarray)
120+
if item.name in dataarray.dims:
121+
dataarray.coords[item.name] = item(dataarray)
122122

123123
for item in model.coord.values():
124-
if item.key not in dataarray.dims:
125-
dataarray.coords[item.key] = item(dataarray)
124+
if item.name not in dataarray.dims:
125+
dataarray.coords[item.name] = item(dataarray)
126126

127127
for item in model.attr.values():
128-
dataarray.attrs[item.key] = item()
128+
dataarray.attrs[item.name] = item()
129129

130130
if model.name:
131131
item = next(iter(model.name.values()))

xarray_dataclasses/datamodel.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@
3737
class Data:
3838
"""Field model for data-related fields."""
3939

40-
key: Hashable
40+
name: Hashable
4141
"""Name of the field."""
4242

4343
value: Any
@@ -90,7 +90,7 @@ def from_field(cls, field: Field[Any], value: Any, of: bool) -> "Data":
9090
class General:
9191
"""Field model for general fields."""
9292

93-
key: Hashable
93+
name: Hashable
9494
"""Name of the field."""
9595

9696
value: Any

xarray_dataclasses/dataset.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -116,18 +116,18 @@ def asdataset(
116116
dataset = dataoptions.factory()
117117

118118
for item in model.data.values():
119-
dataset[item.key] = item(reference)
119+
dataset[item.name] = item(reference)
120120

121121
for item in model.coord.values():
122-
if item.key in dataset.dims:
123-
dataset.coords[item.key] = item(dataset)
122+
if item.name in dataset.dims:
123+
dataset.coords[item.name] = item(dataset)
124124

125125
for item in model.coord.values():
126-
if item.key not in dataset.dims:
127-
dataset.coords[item.key] = item(dataset)
126+
if item.name not in dataset.dims:
127+
dataset.coords[item.name] = item(dataset)
128128

129129
for item in model.attr.values():
130-
dataset.attrs[item.key] = item()
130+
dataset.attrs[item.name] = item()
131131

132132
return dataset
133133

0 commit comments

Comments
 (0)