Skip to content

Revert "Remove unnecessary dtype conversion in icechunk writer"#805

Merged
maxrjones merged 2 commits intomainfrom
revert-760-fix-unnecessary-dtype-conversion
Nov 3, 2025
Merged

Revert "Remove unnecessary dtype conversion in icechunk writer"#805
maxrjones merged 2 commits intomainfrom
revert-760-fix-unnecessary-dtype-conversion

Conversation

@TomNicholas
Copy link
Member

Reverts #760, as per #804

Copy link
Member

@maxrjones maxrjones left a comment

Choose a reason for hiding this comment

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

I think we should merge this to fix #804 until we understand better why the Zarr dtypes aren't working as expected.

@TomNicholas
Copy link
Member Author

👍 go for it

@maxrjones maxrjones merged commit 7a13261 into main Nov 3, 2025
15 checks passed
@maxrjones maxrjones deleted the revert-760-fix-unnecessary-dtype-conversion branch November 3, 2025 22:34
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.

2 participants