Skip to content

Conversation

shoyer
Copy link
Member

@shoyer shoyer commented Oct 5, 2025

xref #10657

  • User visible changes (including notable bug fixes) are documented in whats-new.rst

Co-authored-by: Kai Mühlbauer <kmuehlbauer@wradlib.org>
@shoyer shoyer merged commit 6aa48c3 into pydata:main Oct 6, 2025
33 of 36 checks passed
@shoyer shoyer deleted the switch-backend-back branch October 6, 2025 16:25
@veenstrajelmer
Copy link
Contributor

veenstrajelmer commented Oct 7, 2025

This PR changes line 62:

netcdf_engine_order: Sequence[Literal["netcdf4", "h5netcdf", "scipy"]]

However, I think line 92 should also be changed (or instead):
"netcdf_engine_order": ("h5netcdf", "scipy", "netcdf4"),

And then maybe also the docstring:

netcdf_engine_order : sequence, default ['h5netcdf', 'scipy', 'netcdf4']

Not sure why this is not a single source of truth, but in my experience (#10810 (comment)) h5netcdf is still the default engine in xarray 2025.10.0

@shoyer
Copy link
Member Author

shoyer commented Oct 7, 2025 via email

@kmuehlbauer
Copy link
Contributor

Sorry, from my side, too. Missed it during review.

@shoyer
Copy link
Member Author

shoyer commented Oct 7, 2025

See #10828 for the re-do

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.

3 participants