Skip to content

Commit

Permalink
Rename hdf5_type to data_type in DatasetTable (#651)
Browse files Browse the repository at this point in the history
* Rename hdf5_type to data_type in DatasetTable

* Actually use dtype, cleaner table name on IOC side that way
  • Loading branch information
jwlodek authored Nov 14, 2024
1 parent b4216eb commit ec4b7d0
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion src/ophyd_async/fastcs/panda/_table.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ class PandaHdf5DatasetType(StrictEnum):

class DatasetTable(Table):
name: Sequence[str]
hdf5_type: Sequence[PandaHdf5DatasetType]
dtype: Sequence[PandaHdf5DatasetType]


class SeqTrigger(StrictEnum):
Expand Down
2 changes: 1 addition & 1 deletion tests/fastcs/panda/test_hdf_panda.py
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ def check_dir_exits(value, **kwargs):
mock_hdf_panda.data.datasets,
DatasetTable(
name=["x", "y"],
hdf5_type=[PandaHdf5DatasetType.UINT_32, PandaHdf5DatasetType.FLOAT_64],
dtype=[PandaHdf5DatasetType.UINT_32, PandaHdf5DatasetType.FLOAT_64],
),
)

Expand Down
8 changes: 4 additions & 4 deletions tests/fastcs/panda/test_writer.py
Original file line number Diff line number Diff line change
Expand Up @@ -27,11 +27,11 @@
TABLES = [
DatasetTable(
name=[],
hdf5_type=[],
dtype=[],
),
DatasetTable(
name=["x"],
hdf5_type=[PandaHdf5DatasetType.UINT_32],
dtype=[PandaHdf5DatasetType.UINT_32],
),
DatasetTable(
name=[
Expand All @@ -40,7 +40,7 @@
"y_min",
"y_max",
],
hdf5_type=[
dtype=[
PandaHdf5DatasetType.UINT_32,
PandaHdf5DatasetType.FLOAT_64,
PandaHdf5DatasetType.FLOAT_64,
Expand Down Expand Up @@ -82,7 +82,7 @@ def check_dir_exits(value, **kwargs):
mock_panda.data.datasets,
DatasetTable(
name=[],
hdf5_type=[],
dtype=[],
),
)

Expand Down

0 comments on commit ec4b7d0

Please sign in to comment.