Skip to content
Merged
Show file tree
Hide file tree
Changes from 4 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions CLAUDE.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,12 +31,14 @@ The codebase is a three-language stack. Data and type information flow across al
There is no build step — it's interpreted MATLAB plus a Python module on the path.

**Run the full test suite** (from the `test/` directory, since tests resolve data paths relative to `pwd`):

```matlab
cd test
results = runtests('IncludeSubfolders', true)
```

**Run a single test class or method:**

```matlab
cd test
runtests('tZarrRead') % one class
Expand Down
10 changes: 5 additions & 5 deletions SECURITY.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# Reporting Security Vulnerabilities
# Reporting Security Vulnerabilities

If you believe you have discovered a security vulnerability, please report it to
[security@mathworks.com](mailto:security@mathworks.com). Please see
[MathWorks Vulnerability Disclosure Policy for Security Researchers](https://www.mathworks.com/company/aboutus/policies_statements/vulnerability-disclosure-policy.html)
for additional information.
If you believe you have discovered a security vulnerability, please report it to
[security@mathworks.com](mailto:security@mathworks.com).

Please see: [MathWorks Vulnerability Disclosure Policy for Security Researchers](https://www.mathworks.com/company/aboutus/policies_statements/vulnerability-disclosure-policy.html) for additional information.
5 changes: 2 additions & 3 deletions ZarrDatatype.m
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
%ZARRDATATYPE Datatype of Zarr data
% Represents the datatype mapping between MATLAB, Tensorstore, and Zarr

% Copyright 2025 The MathWorks, Inc.
% Copyright 2025-2026 The MathWorks, Inc.

properties(Constant, Hidden)
% Same-length arrays that represent mapping between
Expand All @@ -28,8 +28,7 @@
MATLABType
end

methods (Hidden)
% "Private" constructor - should not be used directly.
methods (Access = private)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I vaguely remember there being some issue with making this constructor private, but can't find anything about it now. So if everything seems to work with this change, should be all good!

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests pass, and I couldn't find any usages of this constructor outside of the class.

% Use from*Type() static methods instead.
function obj = ZarrDatatype(ind)
obj.Index = ind;
Expand Down
12 changes: 11 additions & 1 deletion test/tZarrRead.m
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,17 @@ function nonExistentArray(testcase)
function tooBigArray(testcase)
% Verify zarrread error when a user tries to read data that is
% too large


% Lower the array size limit so the ~37 GiB array below always
% exceeds it, regardless of how much RAM this machine has.
s = settings;
limitEnabled = s.matlab.desktop.workspace.ArraySizeLimitEnabled;
limitPercent = s.matlab.desktop.workspace.ArraySizeLimit;
testcase.addTeardown(@() clearTemporaryValue(limitEnabled));
testcase.addTeardown(@() clearTemporaryValue(limitPercent));
limitEnabled.TemporaryValue = true;
limitPercent.TemporaryValue = 1;

bigDataPath = "bigData/myzarr";
zarrcreate(bigDataPath, [100000,100000], Datatype='single');
errID = 'MATLAB:Zarr:OutOfMemory';
Expand Down
Loading