Skip to content

Commit a7a698d

Browse files
dstansbynormanrz
andauthored
Fix minimum numpy pin (#628)
* Fix minimum numpy pin * Make sure we test minimum version of numpy * swap import order --------- Co-authored-by: Norman Rzepka <[email protected]>
1 parent 35bd9e3 commit a7a698d

File tree

3 files changed

+6
-3
lines changed

3 files changed

+6
-3
lines changed

.github/workflows/ci.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,9 @@ jobs:
7272
7373
- name: Install zarr-python
7474
shell: "bash -l {0}"
75+
# Since zarr v3 requires numpy >= 1.25, on Python 3.11 leave it out
76+
# so we can have some tests of our minimum version of numpy (1.24)
77+
if: matrix.python-version != '3.11'
7578
run: |
7679
conda activate env
7780
# TODO: remove --pre option when zarr v3 is out

numcodecs/tests/test_zarr3.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,10 @@
33
import numpy as np
44
import pytest
55

6-
import numcodecs.zarr3
7-
86
zarr = pytest.importorskip("zarr")
97

8+
import numcodecs.zarr3 # noqa: E402
9+
1010
pytestmark = [
1111
pytest.mark.skipif(zarr.__version__ < "3.0.0", reason="zarr 3.0.0 or later is required"),
1212
pytest.mark.filterwarnings("ignore:Codec 'numcodecs.*' not configured in config.*:UserWarning"),

pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ A Python package providing buffer compression and transformation codecs \
1515
for use in data storage and communication applications."""
1616
readme = "README.rst"
1717
dependencies = [
18-
"numpy>=1.23",
18+
"numpy>=1.24",
1919
]
2020
requires-python = ">=3.11"
2121
dynamic = [

0 commit comments

Comments
 (0)