Skip to content

refactor warnings #3098

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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
d-v-b merged 31 commits into from
Aug 5, 2025
Merged

refactor warnings #3098

Changes from 1 commit
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
79dd9e2
define zarr-specific FutureWarning and DeprecationWarning
d-v-b May 25, 2025
b2e25cf
Merge branch 'main' of github.com:zarr-developers/zarr-python into re…
d-v-b Jun 30, 2025
7c644eb
make unstablespecificationwarning an instance of zarrfuturewarning
d-v-b Jun 30, 2025
c604075
use pytest.warns instead of pytest.raises
d-v-b Jun 30, 2025
b964ff6
Merge branch 'main' into refactor-warnings
d-v-b Jul 11, 2025
12bea9c
Merge branch 'main' into refactor-warnings
d-v-b Jul 11, 2025
8267c0c
Merge branch 'main' into refactor-warnings
d-v-b Jul 11, 2025
702ef09
Merge branch 'main' of https://github.com/zarr-developers/zarr-python…
d-v-b Jul 31, 2025
64e1405
changelog
d-v-b Jul 31, 2025
d322cad
ensure that all deprecations are ZarrDeprecations
d-v-b Aug 1, 2025
31e63aa
add docstrings and export warnings
d-v-b Aug 1, 2025
917fd3d
fix imports
d-v-b Aug 1, 2025
9055d3b
handle warnings in tests explicitly; make userwarnings ZarrUserWarning
d-v-b Aug 1, 2025
fee48d3
fix doctests by making them more realistic
d-v-b Aug 1, 2025
6a208eb
lint and fix typo
d-v-b Aug 1, 2025
8a2bb9e
Merge branch 'main' into refactor-warnings
d-v-b Aug 1, 2025
a235660
move unstable spec warning to errors
d-v-b Aug 1, 2025
8afdce9
Merge branch 'refactor-warnings' of https://github.com/d-v-b/zarr-pyt…
d-v-b Aug 1, 2025
33a9184
handle warnings in gpu tests
d-v-b Aug 1, 2025
fc72752
handle more warnings
d-v-b Aug 1, 2025
42b4cb5
handle another warning
d-v-b Aug 1, 2025
713bda9
Fix exports
d-v-b Aug 1, 2025
f483eb8
Update src/zarr/errors.py
d-v-b Aug 2, 2025
d029800
Update changes/3098.misc.rst
d-v-b Aug 2, 2025
94ab461
Merge branch 'main' into refactor-warnings
d-v-b Aug 4, 2025
c11328b
Merge branch 'main' into refactor-warnings
d-v-b Aug 4, 2025
fd76671
Merge branch 'main' into refactor-warnings
d-v-b Aug 4, 2025
b7861b7
Merge branch 'main' into refactor-warnings
d-v-b Aug 5, 2025
ba5be4f
Update src/zarr/errors.py
d-v-b Aug 5, 2025
df0eef4
add test to ensure that ambiguous group.open warns
d-v-b Aug 5, 2025
61ff062
update changelog
d-v-b Aug 5, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
handle another warning
  • Loading branch information
@d-v-b
d-v-b committed Aug 1, 2025
commit 42b4cb555750024d1de80b8bca46a3be17567d38
9 changes: 8 additions & 1 deletion tests/test_metadata/test_consolidated.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -497,7 +497,14 @@ async def test_to_dict_order(
await child.create_array("d", shape=(1,), dtype=dtype)

# Consolidate metadata and re-open store
await zarr.api.asynchronous.consolidate_metadata(memory_store)
if zarr_format == 3:
with pytest.warns(
ZarrUserWarning,
match="Consolidated metadata is currently not part in the Zarr format 3 specification.",
):
await zarr.api.asynchronous.consolidate_metadata(memory_store)
else:
await zarr.api.asynchronous.consolidate_metadata(memory_store)
g2 = await zarr.api.asynchronous.open_group(store=memory_store)

assert list(g2.metadata.consolidated_metadata.metadata) == ["a", "b", "c"]
Expand Down
Loading