Skip to content

Commit

Permalink
feat(memh5): remove byte order check in 'deep_group_copy'
Browse files Browse the repository at this point in the history
This was originally required to fix a bug in mpi4py/mpi4py#177
This bug has now been fixed in mpi4py/mpi4py#179
  • Loading branch information
ljgray committed Jul 29, 2024
1 parent 8c0003f commit 13bf29f
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions caput/memh5.py
Original file line number Diff line number Diff line change
Expand Up @@ -2655,12 +2655,6 @@ def _prepare_dataset(dset):
# Unicode characters before writing
data = check_unicode(entry)

if not to_file:
# reading from h5py can result in arrays with explicit endian set
# which mpi4py cannot handle when Bcasting memh5.Group
# needed until fixed: https://github.com/mpi4py/mpi4py/issues/177
data = ensure_native_byteorder(data)

dset_args = {"dtype": data.dtype, "shape": data.shape, "data": data}
# If we're copying memory to memory we can allow distributed datasets
if not to_file and isinstance(dset, MemDatasetDistributed):
Expand Down

0 comments on commit 13bf29f

Please sign in to comment.