Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

align_chunks not working for datasets#10516

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
josephnowak wants to merge16 commits intopydata:main
base:main
Choose a base branch
Loading
fromjosephnowak:fix/align-chunks

Conversation

josephnowak
Copy link
Contributor

@josephnowakjosephnowak commentedJul 8, 2025
edited
Loading

I forgot to pass the align_chunks to the to_zarr method on the datasets, which makes the feature useless for this kind of data structure. I added a specific test to cover this issue.

Now, the align_chunks works on the cases where the data is smaller than a single Zarr chunk (also, one test was added to cover this scenario).

I modified (again) the error message shown with the safe_chunks, now it includes information about the two chunks that overlap with a single Zarr chunk. From what I saw on the error 10501, the original message was not helping the users to understand what was happening.

@josephnowakjosephnowak marked this pull request as draftJuly 9, 2025 14:15
@josephnowak
Copy link
ContributorAuthor

Looks like those tests failing are not related to this PR

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

to_zarr() via dask with silent data loss on append
2 participants
@josephnowak@dcherian

[8]ページ先頭

©2009-2025 Movatter.jp