Skip to content

Issue #1931 fix stringdtype conversion - #1932

Merged
JoerivanEngelen merged 10 commits into
masterfrom
issue_#1931_fix_stringdtype_conversion
Oct 12, 2026
Merged

JoerivanEngelen merged 10 commits into
masterfrom
issue_#1931_fix_stringdtype_conversion

Conversation

@JoerivanEngelen

@JoerivanEngelen JoerivanEngelen commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Fixes #1931

Description

Fixes breaking CI after lockfile update. An update in xarray triggers an error when reading string data from a zarr file. This introduces a temporary workaround which can be removed once numpy 2.6.0 is released (for which I didn't see any release date).

Checklist

  • Links to correct issue
  • Update changelog, if changes affect users
  • PR title starts with Issue #nr, e.g. Issue #737
  • Unit tests were added
  • If feature added: Added/extended example
  • If feature added: Added feature to API documentation
  • If pixi.lock was changed: Ran pixi run generate-sbom and committed changes

@JoerivanEngelen
JoerivanEngelen added this pull request to stack #1933 October 12, 2026 13:25
@JoerivanEngelen
JoerivanEngelen force-pushed the issue_#1931_fix_stringdtype_conversion branch from e5b8748 to 267ab2d Compare October 12, 2026 13:25

@LuukBlom LuukBlom left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM,

I think the function could do with a slightly more descriptive name, but its also a bit nitpicky so up to you!

Comment thread imod/common/utilities/dtype.py Outdated

@ClaireDons ClaireDons left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, do you have a reminder somewhere to remove this when the numpy fix comes out?

@JoerivanEngelen

JoerivanEngelen commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor Author

Looks good, do you have a reminder somewhere to remove this when the numpy fix comes out?

Yes, I want to create a follow-up issue once this was merged.

@sonarqubecloud

Copy link
Copy Markdown

Base automatically changed from pixi_update_162 to master October 12, 2026 15:03
@JoerivanEngelen
JoerivanEngelen merged commit be07b04 into master Oct 12, 2026
8 checks passed
@JoerivanEngelen
JoerivanEngelen deleted the issue_#1931_fix_stringdtype_conversion branch October 12, 2026 15:03
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.

[Bug] - Xarray/numpy update causes dump/from_file roundtrip to fail for zarr files

3 participants