Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clarify use of _ARRAY_DIMENSIONS in v0.4 and latest #218

Merged
merged 3 commits into from
Oct 18, 2023
Merged

Clarify use of _ARRAY_DIMENSIONS in v0.4 and latest #218

merged 3 commits into from
Oct 18, 2023

Conversation

tcompa
Copy link
Contributor

@tcompa tcompa commented Oct 18, 2023

As of version 0.4 (and then including latest version), _ARRAY_DIMENSIONS is not a required attribute any more. This PR replaces the relevant "must" with a "may", and adds a clarification about the goal of this attribute (that is, it exists in view of xarray compatibility).

Closes #217

@github-actions
Copy link
Contributor

Automated Review URLs

@will-moore
Copy link
Member

Looks good rendered at http://api.csswg.org/bikeshed/?url=https://raw.githubusercontent.com/ome/ngff/3fca4de96356213d9538dcc8bef688385c2bc703/latest/index.bs

Readthedocs build failure above looks unrelated to this PR - will open a PR to fix that if I can find a solution...

@will-moore will-moore merged commit 6775694 into ome:main Oct 18, 2023
7 of 8 checks passed
github-actions bot added a commit that referenced this pull request Oct 18, 2023
Clarify use of `_ARRAY_DIMENSIONS` in v0.4 and latest

SHA: 6775694
Reason: push, by @will-moore

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clarification about _ARRAY_DIMENSIONS
2 participants