mathworks / mathworks/MATLAB-support-for-Zarr-files
Improvement to the error message in case of size mismatch between the data to be written and the shape
Open
Nobody has claimed this yet.
enhancement
- Dominant language
- MATLAB
- Stars
- 19
- Forks
- 3
- PR merge metrics
- No merged PRs in 30d
Description
The current error message looks a bit incomplete,
>> zarrwrite("mygrp/arr1",ones(10,10,3))
Error using Zarr/write (line 150)
Size of the data to be written does not match.
Maybe this can be be updated to something like, "Size of the data to be written does not match the shape of the array"?
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Reproduce the size-mismatch example with zarrwrite("mygrp/arr1",ones(10,10,3)), then inspect the error raised from Zarr/write at line 150. Update the message so it explains that the data size does not match the array shape, and confirm the example reports the clearer wording.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- matlab
- Domain
- data
- Issue type
- Bug
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 55/100