fix BloscCodec with shuffle = "shuffle"

scalableminds · Jun 3, 2024 · 39ffce5 · 39ffce5
1 parent db47786
commit 39ffce5
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/src/main/java/dev/zarr/zarrjava/v3/codec/CodecBuilder.java b/src/main/java/dev/zarr/zarrjava/v3/codec/CodecBuilder.java
@@ -41,6 +41,9 @@ public CodecBuilder withBlosc(
   }
 
   public CodecBuilder withBlosc(String cname, String shuffle, int clevel, int blockSize) {
+    if (shuffle.equals("shuffle")){
+      shuffle = "byteshuffle";
+    }
     return withBlosc(Blosc.Compressor.fromString(cname), Blosc.Shuffle.fromString(shuffle), clevel,
         dataType.getByteCount(), blockSize
     );

diff --git a/src/main/java/dev/zarr/zarrjava/v3/codec/core/BloscCodec.java b/src/main/java/dev/zarr/zarrjava/v3/codec/core/BloscCodec.java
@@ -84,7 +84,7 @@ public void serialize(Blosc.Shuffle shuffle, JsonGenerator generator,
           generator.writeString("bitshuffle");
           break;
         case BYTE_SHUFFLE:
-          generator.writeString("byteshuffle");
+          generator.writeString("shuffle");
           break;
       }
     }
@@ -154,7 +154,7 @@ public Blosc.Shuffle deserialize(JsonParser jsonParser, DeserializationContext c
           return Blosc.Shuffle.NO_SHUFFLE;
         case "bitshuffle":
           return Blosc.Shuffle.BIT_SHUFFLE;
-        case "byteshuffle":
+        case "shuffle":
           return Blosc.Shuffle.BYTE_SHUFFLE;
         default:
           throw new JsonParseException(