Skip to content

Add type annotations to open_zarr#11226

Open
wavebyrd wants to merge 3962 commits intopydata:mainfrom
wavebyrd:fix-open-zarr-typing
Open

Add type annotations to open_zarr#11226
wavebyrd wants to merge 3962 commits intopydata:mainfrom
wavebyrd:fix-open-zarr-typing

Conversation

@wavebyrd
Copy link

@wavebyrd wavebyrd commented Mar 13, 2026

Adds type annotations to all parameters of open_zarr and its return type.

Previously the function was essentially untyped, which caused type checkers like pyright to reject valid arguments such as chunks="auto" because the default value _default (of type Default) was the only inferred type.

  • chunks is now typed as T_Chunks | Default
  • All other parameters annotated to match their documented types
  • Return type annotated as Dataset

Fixes #11221

dhruvak001 and others added 30 commits August 25, 2025 15:35
@dhruvak001 @pre-commit-ci @dcherian
Support rechunking to seasonal frequency with SeasonalResampler (pyda…
d7feece 
…ta#10519)

Co-authored-by: DHRUVA KUMAR KAUSHAL <sanjay@MacBook-Air.local>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Deepak Cherian <deepak@cherian.net>
@keewis
migrate to xarray-contrib/minimum-dependency-versions (pydata#10677)
fb0ce2f 
* create a policy definition file

* use `xarray-contrib/minimum-dependency-versions`

* remove the min-deps script

* setup python

* bump versions

* exclude `hypothesis`

* use `python-version` to select the version

* stop ignoring `pydap` violations

We appear to have reached the point where this is not necessary
anymore (but the comment might have been wrong? We're already pinning to
`pydap=3.5.0`)
@spencerkclark
Following pandas 3.0, make Day cftime offset non-Tick-like (pydat…
1603ae8 
…a#10650)

In pandas 3.0, `Day` will no longer be considered a `Tick`-like frequency (pandas-dev/pandas#61985). This PR ports this change to the cftime version of this offset. The main implication in the cftime / xarray context is that it means the `offset` and `origin` options in `resample` will no longer have an effect when resampling to a daily frequency. As in pandas-dev/pandas#62101, warnings will be emitted if non-default values are passed.
@Illviljan @pre-commit-ci
Use better contour defaults with logarithmic norms (pydata#10565)
43b5d5a 
* Remove default re-definition

* colors arg should take priority over default cmaps

* Update test_plot.py

* Update test_plot.py

* add log norm test

* Add tests

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Update test_plot.py

* Update test_plot.py

* Update test_plot.py

* Update test_plot.py

* Update whats-new.rst

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@shoyer
Improve consistency of default engine and return memoryview instead o…
93eb3e6 
…f bytes from to_netcdf() (pydata#10656)

* Improve consistency and engine keyword argument for to_netcdf()

This PR introduces a bug fix and a breaking changes:

1. The default backend ``engine`` used by `Dataset.to_netcdf`
   and `DataTree.to_netcdf` is now chosen consistently with
   `open_dataset` and `open_datatree`, using whichever netCDF
   libraries are available and preferring netCDF4 to h5netcdf to scipy.
   Previously, `DataTree.to_netcdf` was hard-coded to use h5netcdf.
2. The return value of `Dataset.to_netcdf` without ``path`` is
   now a ``memoryview`` object instead of ``bytes``. This removes an unnecessary
   memory copy and ensures consistency when using either ``engine="scipy"`` or
   ``engine="h5netcdf"``.

Fixes pydata#10654

* Add PR number to whatsnew

* Consistently use BytesIOProxy

* Fix test_engine

* Clarify whats new

* Clarify whats new

* tweak whats-new

* Fix release note typo

* Rename to_file_or_memoryview

* test fixes
@pre-commit-ci @shoyer
Update pre-commit hooks (pydata#10681)
846be94 
updates:
- [github.com/pre-commit/pre-commit-hooks: v5.0.0 → v6.0.0](pre-commit/pre-commit-hooks@v5.0.0...v6.0.0)
- [github.com/astral-sh/ruff-pre-commit: v0.12.7 → v0.12.11](astral-sh/ruff-pre-commit@v0.12.7...v0.12.11)
- [github.com/adhtruong/mirrors-typos: v1.34.0 → v1.35.6](adhtruong/mirrors-typos@v1.34.0...v1.35.6)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Stephan Hoyer <shoyer@google.com>
@max-sixty
Allow tests to pass locally on Mac (pydata#10687)
7887289 
* Allow tests to pass locally on Mac

move env vars into the tests, out of the env setup. xfail tests without codec dependencies (I think this is a PyPI vs conda diff?)

* Fix ruff linting issue: use contextlib.suppress instead of try-except-pass
@max-sixty @claude
Fix mypy errors in test_backends.py (pydata#10689)
938f469 
- Add explicit keyword arguments to netCDF4.Dataset.createVariable calls
- Add type: ignore comments for dynamic compression parameter
- Remove unnecessary type: ignore for requests import

These changes resolve mypy type checking errors that were causing CI failures.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-authored-by: Claude <noreply@anthropic.com>
@max-sixty @claude
Remove obsolete warning exclusions from pyproject.toml (pydata#10690)
8dcc56e 
Removed 11 warning exclusions that are no longer needed:
- Invalid cast warnings from duck_array_ops and test_array_api
- CachingFileManager deallocation warnings from backends
- Deprecated treenode methods (ancestors, iter_lineage, lineage)
- Test-specific deprecations that no longer exist

These exclusions were verified to be safe to remove through testing.
The test suite passes with 20,779 tests after removal.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-authored-by: Claude <noreply@anthropic.com>
@max-sixty @claude
Enable mypy checking for test_utils and test_parallelcompat (pydata#1…
bda0afa 
…0691)

* Enable mypy checking for test_utils and test_parallelcompat

Removes xarray.tests.test_utils and xarray.tests.test_parallelcompat from the mypy
exclusion list by fixing type issues in these files:

- Added type annotations where needed
- Added targeted type: ignore comments for intentional test behaviors
- Fixed function call signatures to match expected types

All tests continue to pass and mypy now checks these files with check_untyped_defs=true.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Also enable mypy for test_missing

Removes xarray.tests.test_missing from the mypy exclusion list by
fixing type issues:

- Added type: ignore comments for intentional test cases with invalid methods
- Added type annotation for kwargs dict

All 115 tests continue to pass.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

---------

Co-authored-by: Claude <noreply@anthropic.com>
@max-sixty @claude
Add CLAUDE.md with development setup instructions (pydata#10692)
128e4f5 
Simple development instructions for using uv with xarray, including
commands for running tests and linting.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-authored-by: Claude <noreply@anthropic.com>
@max-sixty @claude @pre-commit-ci
Further warning exclusion cleanup and test improvements (pydata#10693)
2f83c68 
* Remove obsolete warning exclusions from pyproject.toml

Removed 11 warning exclusions that are no longer needed:
- Invalid cast warnings from duck_array_ops and test_array_api
- CachingFileManager deallocation warnings from backends
- Deprecated treenode methods (ancestors, iter_lineage, lineage)
- Test-specific deprecations that no longer exist

These exclusions were verified to be safe to remove through testing.
The test suite passes with 20,779 tests after removal.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Remove additional obsolete UserWarning exclusions

Removed 3 more warning exclusions that are no longer needed:
- UserWarning from test_coding_times
- UserWarning from test_computation
- UserWarning from test_dataset

All test files pass without these warning exclusions.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Remove 3 more obsolete warning exclusions

Removed warning exclusions that are no longer needed:
- "No index created" UserWarning - Tests properly handle with pytest.warns
- "pandas.MultiIndex" FutureWarning - No longer triggered
- "Duplicate dimension names" UserWarning - Tests handle with local filterwarnings

These warnings are either properly tested or no longer occur.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Use pytest.warns consistently for quantile interpolation deprecation

Fixed test_dataset.py to use pytest.warns instead of warnings.catch_warnings
for testing the interpolation->method deprecation warning. This makes it
consistent with the other test files.

Note: We cannot remove the global warning exclusion because the error:::xarray.*
rule converts warnings to errors before pytest.warns can catch them. This is
a known limitation of the current filterwarnings configuration.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Add local filterwarnings to quantile interpolation deprecation tests

Instead of relying on a global warning exclusion, added
@pytest.mark.filterwarnings decorators to the specific tests that
test the interpolation->method deprecation warning. This allows
the warning to be properly tested while avoiding the conflict
with the error:::xarray.* rule.

Now the global exclusion for this warning can be safely removed.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@shoyer
Fix unreleased bug in DataTree.to_zarr() with mode='r+' (pydata#10685)
1e20a32
@ianhi
QOL: detect both .zarr and .zarr/ as using the zarr engine (pydat…
d8f1eed 
…a#10697)

* QOL: detect both .zarr and .zarr/ as using the zarr engine

* test: properly test the opening with a trailing slash
@max-sixty @claude @pre-commit-ci
Remove most warning exclusions (pydata#10695)
bbbb759 
* Remove obsolete warning exclusions from pyproject.toml

Removed 11 warning exclusions that are no longer needed:
- Invalid cast warnings from duck_array_ops and test_array_api
- CachingFileManager deallocation warnings from backends
- Deprecated treenode methods (ancestors, iter_lineage, lineage)
- Test-specific deprecations that no longer exist

These exclusions were verified to be safe to remove through testing.
The test suite passes with 20,779 tests after removal.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Remove additional obsolete UserWarning exclusions

Removed 3 more warning exclusions that are no longer needed:
- UserWarning from test_coding_times
- UserWarning from test_computation
- UserWarning from test_dataset

All test files pass without these warning exclusions.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Remove 3 more obsolete warning exclusions

Removed warning exclusions that are no longer needed:
- "No index created" UserWarning - Tests properly handle with pytest.warns
- "pandas.MultiIndex" FutureWarning - No longer triggered
- "Duplicate dimension names" UserWarning - Tests handle with local filterwarnings

These warnings are either properly tested or no longer occur.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Use pytest.warns consistently for quantile interpolation deprecation

Fixed test_dataset.py to use pytest.warns instead of warnings.catch_warnings
for testing the interpolation->method deprecation warning. This makes it
consistent with the other test files.

Note: We cannot remove the global warning exclusion because the error:::xarray.*
rule converts warnings to errors before pytest.warns can catch them. This is
a known limitation of the current filterwarnings configuration.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Add local filterwarnings to quantile interpolation deprecation tests

Instead of relying on a global warning exclusion, added
@pytest.mark.filterwarnings decorators to the specific tests that
test the interpolation->method deprecation warning. This allows
the warning to be properly tested while avoiding the conflict
with the error:::xarray.* rule.

Now the global exclusion for this warning can be safely removed.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Remove non-tuple sequence indexing warning exclusion

Fixed the test in test_nputils.py that was using deprecated list indexing
x[[1, 2], [1, 2]] by changing it to tuple indexing x[([1, 2], [1, 2])].

This allows us to remove the global warning filter for 'Using a non-tuple
sequence for multidimensional indexing is deprecated'.

* Remove two obsolete NumPy DeprecationWarning exclusions

Removed the following warning exclusions from pyproject.toml:
- 'Failed to decode variable.*NumPy will stop allowing conversion...'
- 'NumPy will stop allowing conversion of:DeprecationWarning'

These exclusions are no longer needed as the tests pass without them.
The remaining 'invalid value encountered in cast' warnings are legitimate
and occur when casting NaN values to integer types.

* Change Zarr V3 warnings to default

These warnings are no longer ignored and will be reported in CI.

Co-authored-by: Claude <no-reply@anthropic.com>

* Remove 'invalid value encountered in cast' warning exclusions

Fixed tests to properly handle the expected RuntimeWarning when casting
NaN values to integer types:

- Updated test_conventions.py::test_missing_fillvalue to explicitly catch
  both the SerializationWarning and the numpy RuntimeWarning
- Added create_nan_array() helper function in test_units.py that suppresses
  the cast warning when creating test arrays with NaN values for int dtypes
- Removed the two warning exclusions from pyproject.toml

These warnings were legitimate - they occur when casting float arrays
containing NaN to integer types, which is expected behavior in these
test scenarios.

* Fix NumPy out-of-bound integer conversion warning

Handle overflow when casting _FillValue to dtype in CFMaskCoder.encode().
This fixes CI failures on older NumPy versions where casting 255 to int8
raises a DeprecationWarning (newer NumPy raises OverflowError).

The fix:
- Wraps the dtype.type(fv) call in a try/except block
- Suppresses the NumPy DeprecationWarning for older versions
- Catches OverflowError for newer NumPy versions
- Uses np.array(fv).astype(dtype) which properly wraps the value

* Fix remaining NumPy out-of-bound integer conversion warnings

Added comprehensive handling for NumPy's DeprecationWarning about
out-of-bound integer conversion in multiple locations:

- Added _safe_type_cast() helper function to handle the conversion safely
- Updated _encode_unsigned_fill_value() to suppress the warning
- Fixed missing_value encoding to use _safe_type_cast()
- Refactored _FillValue encoding to use the helper function

This should fix all test failures in the bare-min-and-scipy CI environment
where older NumPy versions raise DeprecationWarning instead of OverflowError.

* Simplify NumPy overflow handling with cleaner approach

Replaced complex warning suppression with a simpler, more consistent approach:
- _safe_type_cast() now uses np.array(value).astype(dtype).item() which works
  consistently across NumPy 1.x and 2.x for overflow cases
- _encode_unsigned_fill_value() now explicitly checks bounds using np.iinfo()
  before attempting the cast, making the logic clearer
- This removes unnecessary try/except blocks and warning filters

The root issue: NumPy changed behavior between versions:
- NumPy 1.x: dtype.type(out_of_bounds) raises DeprecationWarning but succeeds
- NumPy 2.x: dtype.type(out_of_bounds) raises OverflowError

The test itself (test_roundtrip_unsigned) is correctly testing edge cases
where unsigned values (255) need to be stored as signed int8, which is a
legitimate use case in CF conventions for unsigned integer encoding.

* Add autouse fixture for NumPy 1.x warning handling

- Added handle_numpy_1_warnings autouse fixture to conftest.py
- Removes need for workarounds in the actual code
- Handles NumPy version differences cleanly at the test level
- Reverted variables.py to simpler implementation without _safe_type_cast

---------

Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Claude <no-reply@anthropic.com>
@max-sixty @claude @pre-commit-ci
Upgrade mypy to 1.17.1 and fix all type errors (pydata#10694)
efd68c1 
* Upgrade mypy to 1.17.1 and fix all type errors

- Update mypy from 1.15 to 1.17.1 in CI requirements
- Fix 65 mypy errors across 20 files to achieve zero errors
- Key fixes:
  - Remove obsolete type: ignore comments
  - Fix pandas MultiIndex type compatibility (list conversions)
  - Resolve dictionary key type issues (Hashable vs str)
  - Add type annotations for better type inference
  - Fix scipy/pandas interface compatibility in tests
  - Correct Variable.data setter override placement

All changes are minimal type annotation improvements without logic modifications.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Fix test_stack_unstack for pandas compatibility

Keep pd.Index objects instead of converting to lists to preserve dtype behavior in MultiIndex levels.

* Add unused-ignore to all type: ignore comments for CI compatibility

CI environment may report some type: ignore comments as unused while they're needed locally. Adding unused-ignore to suppress these warnings.

* Add types-requests to CI and remove unnecessary type: ignore comments

- Added types-requests to CI dependencies to provide typing for requests module
- Removed type: ignore comments that were showing as unused both locally and in CI
- Kept only the type: ignore that are actually necessary (one in test_plot.py)

* Add back necessary type: ignore comments for CI compatibility

The CI environment showed different mypy behavior than local environment,
requiring us to keep these type: ignore comments with unused-ignore flag

* Fix remaining CI mypy errors

- Add type: ignore for indexes.py indexing operation
- Add type: ignore for alignment.py return type issues
- Add type: ignore for cftimeindex.py shift method override

* Improve level_coords_dtype typing to match pandas MultiIndex

- Change level_coords_dtype type from dict[str, Any] to dict[Hashable | None, Any]
- This matches pandas MultiIndex.names which can be Hashable or None
- Remove unnecessary str() conversions added in previous commit
- Remove obsolete type: ignore comment and TODO that questioned if Hashables are ok (they are)

This is a cleaner solution that better reflects the actual pandas API.

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Fix IndexVariable.level_names type to match pandas

- Changed return type from list[str] | None to list[Hashable | None] | None
- This matches pandas MultiIndex.names actual type (list[Hashable | None])
- Removed unnecessary str() conversion that was losing type information
- Now returns list(index.names) directly, preserving the correct types

---------

Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@shoyer @max-sixty
Release notes for 2025.09.0 (pydata#10684)
2f8d50a 
Co-authored-by: Maximilian Roos <5635139+max-sixty@users.noreply.github.com>
@shoyer
empty whats new (pydata#10699)
f87c5a0
@shoyer
Update instructions for updating pyodide (pydata#10701)
944ff2d
@keewis
add a new pattern to the non-existent store patterns (pydata#10700)
d5ca917 
* ignore the new pattern

* construct the pattern dynamically
@ianhi
fix: don't destroy dtypes in to_dataframe (pydata#10705)
c0881f0
@TomNicholas @pre-commit-ci
Update url for example h5netcdf ros3 dataset (pydata#10713)
3288fd9 
* update url

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@max-sixty @claude
Add GitHub interaction guidelines to CLAUDE.md (pydata#10715)
58200b7 
Added explicit guidelines about GitHub interactions to ensure appropriate behavior when working with the xarray repository. These guidelines prevent unsolicited GitHub actions and require explicit user authorization for repository interactions.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-authored-by: Claude <noreply@anthropic.com>
@max-sixty @claude
Add types-requests to type dependencies (pydata#10716)
a58c308 
🤖 Generated with [Claude Code](https://claude.ai/code)

Co-authored-by: Claude <noreply@anthropic.com>
@max-sixty @shoyer
Complete deprecation cycle for Dataset.update return value (pydata#10717
835d550 
)

* Complete deprecation cycle for Dataset.update return value

- Dataset.update() now returns None instead of self
- Removes deprecation that was started in v0.17 and scheduled for removal in v0.21
- Updates documentation to remove return value information
- Aligns behavior with standard dict.update() which returns None

Closes pydata#10167

* Update doc/whats-new.rst

Co-authored-by: Stephan Hoyer <shoyer@google.com>

---------

Co-authored-by: Stephan Hoyer <shoyer@google.com>
@ianhi
Add json file for Xarray Badge (pydata#10666)
62ad8ed 
* fun: add badge

Co-authored-by: pierre-manchon <pierre-manchon@users.noreply.github.com>

* fun: add badge to readme

* Update badge.json

* move badge and use more accessible color

---------

Co-authored-by: pierre-manchon <pierre-manchon@users.noreply.github.com>
@VeckoTheGecko
Fix Xarray badge (pydata#10721)
48b0c18
@dependabot
Bump the actions group with 4 updates (pydata#10719)
cb11eab 
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@josephnowak @pre-commit-ci
align_chunks not working for datasets (pydata#10516)
ec3b0d7 
* The align_chunks parameter was not being sent on the to_zarr method of the datasets

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Add a note on the whats-new.rst about the error of the align_chunks for datasets

* Fix a ValueError on the test_dataset_to_zarr_align_chunks_true

* Fix the case when enc_chunks are bigger than the dask chunks

* Linter

* Fix small reintroduced issue when the region is None

* Fix mypy issues

* Update whats-new.rst

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Use "v" instead of "var" to follow the name convention used on the rest of Xarray, move the modification of the enc_chunks to the build_grid_chunks function, add additional test to covert the scenario where the chunk is bigger than the size of the array

* Update the whats-new.rst

* Fix whats-new.rst

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@spencerkclark
Fix bug when encoding an empty datetime64 array (pydata#10723)
f1891a0
VeckoTheGecko and others added 8 commits February 25, 2026 11:56
@VeckoTheGecko
Revert "Add osx-64 to the pixi config." (pydata#11203)
9bd23df 
This reverts commit cf6bf5b.
@BorisTheBrave @jsignell @VeckoTheGecko
Test dependency fixes (pydata#11172)
ff06a4d 
Co-authored-by: Julia Signell <jsignell@gmail.com>
Co-authored-by: Nick Hodgskin <36369090+VeckoTheGecko@users.noreply.github.com>
@yaochengchen @VeckoTheGecko
Fix a typo in dataset.py (pydata#11181)
addb618 
* Fix a typo in dataset.py

* change whats-new.rst

* Update whatsnew

---------

Co-authored-by: Nick Hodgskin <36369090+VeckoTheGecko@users.noreply.github.com>
@dependabot
Bump the actions group with 2 updates (pydata#11208)
9d8c615 
Bumps the actions group with 2 updates: [actions/upload-artifact](https://github.com/actions/upload-artifact) and [actions/download-artifact](https://github.com/actions/download-artifact).


Updates `actions/upload-artifact` from 6 to 7
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v6...v7)

Updates `actions/download-artifact` from 7 to 8
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](actions/download-artifact@v7...v8)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: actions
- dependency-name: actions/download-artifact
  dependency-version: '8'
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: actions
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@jsignell
Fix TestDataArray.test_curvefit_helpers and `TestIndexVariable.test…
b89bf89 
…_concat_periods` (pydata#11204)

* Filter deprecated warning
* Remove inspect.signature raises error test. The example no longer raises an error
* Type of error changed in numpy 2.4
@pre-commit-ci @headtr1ck @jsignell
Update pre-commit hooks (pydata#11211)
c1126de 
* Update pre-commit hooks

updates:
- [github.com/astral-sh/ruff-pre-commit: v0.14.14 → v0.15.4](astral-sh/ruff-pre-commit@v0.14.14...v0.15.4)
- [github.com/abravalheri/validate-pyproject: v0.24.1 → v0.25](abravalheri/validate-pyproject@v0.24.1...v0.25)
- [github.com/adhtruong/mirrors-typos: v1.42.3 → v1.44.0](adhtruong/mirrors-typos@v1.42.3...v1.44.0)

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* apply ruff changes

* fix mypy stuff

* fix stubtest, don't ask me why or how

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Michael Niklas <mick.niklas@gmail.com>
Co-authored-by: Julia Signell <jsignell@gmail.com>
@Harteg
Add TOAD to xarray ecosystem page (pydata#11219)
20bde3e
@wavebyrd
Add type annotations to open_zarr parameters
8d80b04 
All parameters in open_zarr were untyped, which caused type checkers
like pyright to reject valid arguments (e.g. passing "auto" for chunks).

Adds annotations to every parameter and the return type, using T_Chunks
for the chunks parameter and matching types from the docstrings for the
rest.

Fixes pydata#11221
@github-actions github-actions bot added topic-backends topic-zarr Related to zarr storage library io labels Mar 13, 2026
@wavebyrd wavebyrd force-pushed the fix-open-zarr-typing branch from 7889687 to 8d80b04 Compare March 13, 2026 21:39
@github-actions github-actions bot added topic-indexing topic-CF conventions topic-groupby topic-dask topic-plotting topic-testing topic-performance topic-cftime topic-arrays related to flexible array support CI Continuous Integration tools topic-html-repr topic-rolling dependencies Pull requests that update a dependency file Automation Github bots, testing workflows, release automation topic-DataTree Related to the implementation of a DataTree class topic-typing topic-hypothesis Strategies or tests using the hypothesis library topic-NamedArray Lightweight version of Variable labels Mar 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Automation Github bots, testing workflows, release automation CI Continuous Integration tools dependencies Pull requests that update a dependency file io topic-arrays related to flexible array support topic-backends topic-CF conventions topic-cftime topic-dask topic-DataTree Related to the implementation of a DataTree class topic-groupby topic-html-repr topic-hypothesis Strategies or tests using the hypothesis library topic-indexing topic-NamedArray Lightweight version of Variable topic-performance topic-plotting topic-rolling topic-testing topic-typing topic-zarr Related to zarr storage library

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

chunks argument is typed incorrected in open_zarr