updating error messages, clean-up, restricting remote test to >=R2024b

Author: krisfed

.github/workflows/test_setup.yml

@@ -8,7 +8,7 @@ jobs:
     strategy:
       matrix:
         os: [ubuntu-latest, windows-latest, macos-14]
-        matlab-version: ['R2024b']
+        matlab-version: ['R2024a', 'R2024b']
 
     steps:
     - name: Checkout code

Zarr.m

@@ -106,8 +106,8 @@ function pyReloadInProcess()
             if any(params>dims)
                 error("MATLAB:Zarr:PartialReadOutOfBounds",...
                     "Elements in %s must not exceed "+...
-                    "the corresponding Zarr array dimensions (%s).",...
-                    paramName, join(string(dims), "x"))
+                    "the corresponding Zarr array dimensions.",...
+                    paramName)
             end
 
             newParams = params;
@@ -325,8 +325,7 @@ function makeZarrGroups(existingParentPath, newGroupsPath)
             if any(count>maxCount)
                 error("MATLAB:Zarr:PartialReadOutOfBounds",...
                     "Requested Count in combination with other "+...
-                    "partial read parameters exceeds "+...
-                    "Zarr array dimensions.")
+                    "parameters exceeds Zarr array dimensions.")
             end
 
             % Convert partial read parameters to tensorstore-style
@@ -344,19 +343,16 @@ function makeZarrGroups(existingParentPath, newGroupsPath)
                 zeros(count, obj.Datatype.MATLABType);
             catch ME
                 if strcmp(ME.identifier, 'MATLAB:array:SizeLimitExceeded')
-                    %rethrow(ME)
                     error("MATLAB:Zarr:OutOfMemory",...
-                        "Reading requested data (%s %s array) "+...
+                        "Reading requested data (%s %s matrix) "+...
                         "might exceed available memory.",...
-                        join(string(count), "x"), obj.Datatype.MATLABType)
+                        join(string(count), "-by-"), obj.Datatype.MATLABType)
                 end
             end
 
             % Read the data
             ndArrayData = Zarr.ZarrPy.readZarr(obj.KVStoreSchema,...
                 start, endInds, stride);
-
-            % assert(ndArrayData.dtype.name == obj.Datatype.TensorstoreType)
 
             % Convert the numpy array to MATLAB array
             data = cast(ndArrayData, obj.Datatype.MATLABType);

test/tZarrRead.m

@@ -113,6 +113,8 @@ function tooBigArray(testcase)
             % Verify zarrread error when a user tries to read data that is
             % too large
 
+            testcase.assumeFalse(isMATLABReleaseOlderThan('R2024b'),...
+                "Remote workflows are fully supported only from R2024b")
             errID = 'MATLAB:Zarr:OutOfMemory';
             bigData = 'https://noaa-nwm-retro-v2-zarr-pds.s3.amazonaws.com/elevation/';
             testcase.verifyError(@()zarrread(bigData,Count=[100000,100000]),errID);