Fixed zarrs_reencode with --shard-shape doubly applying codecs

LDeakin · Jul 29, 2024 · 7197bf6 · 7197bf6
1 parent a2356ce
commit 7197bf6
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -10,6 +10,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Changed
  - Bump `zarrs` to 0.16.0
 
+### Fixed
+ - Fixed `zarrs_reencode` with `--shard-shape` applying existing array-to-array and bytes-to-bytes codecs as both inner and outer codecs
+
 ## [0.5.3] - 2024-07-24
 
 ### Added

diff --git a/src/lib.rs b/src/lib.rs
@@ -569,12 +569,14 @@ pub fn get_array_builder_reencode<TStorage: ?Sized>(
             array_to_bytes_codec,
             bytes_to_bytes_codecs,
         );
+        array_builder.array_to_array_codecs(vec![]);
         array_builder.array_to_bytes_codec(Box::new(ShardingCodec::new(
             chunk_shape.try_into().unwrap(),
             inner_codecs,
             index_codecs,
             sharding::ShardingIndexLocation::End,
         )));
+        array_builder.bytes_to_bytes_codecs(vec![]);
     } else {
         array_builder.array_to_array_codecs(array_to_array_codecs);
         array_builder.array_to_bytes_codec(array_to_bytes_codec);