Skip to content

Loosen numpy constraint from ==2.2.0.* to ==2.2.*#162

Merged
Gerrrr merged 1 commit into
apache:masterfrom
jordancharest:loosen-numpy-2.2
Jun 2, 2026
Merged

Loosen numpy constraint from ==2.2.0.* to ==2.2.*#162
Gerrrr merged 1 commit into
apache:masterfrom
jordancharest:loosen-numpy-2.2

Conversation

@jordancharest

Copy link
Copy Markdown
Contributor

What

Loosen the numpy version constraint for Python < 3.14 from ==2.2.0.* to ==2.2.*.

Why

The inline comment in pyproject.toml already documents the intent as:

NumPy 2.2.x supports Python 3.10–3.13 [...]

However, ==2.2.0.* is a prefix match that pins to exactly 2.2.0 (numpy publishes no 4-segment versions), so it excludes the 2.2.12.2.6 patch releases along with their bug/security fixes. This looks unintentional: the dependency was a minor-level pin (numpy==1.24.*) until #108 bumped it to 2.2.0.*, where the natural equivalent would have been 2.2.*.

Loosening to ==2.2.* makes the constraint match the documented intent and lets users pick up 2.2.x patch releases, while still excluding 2.3 (which is handled separately for Python ≥ 3.14).

Changes

  • pyproject.toml: numpy==2.2.0.*numpy==2.2.* (Python < 3.14)
  • uv.lock: regenerated via uv lock (numpy 2.2.02.2.6)

Testing

  • uv run pytest (tests + perf): 112 passed against numpy 2.2.6
  • uv run tox -e lint: passed (ruff, autoflake, isort, flake8, license headers)

The inline comment already documents the intent as "NumPy 2.2.x supports
Python 3.10-3.13", but ==2.2.0.* pins to exactly 2.2.0 and excludes the
2.2.1-2.2.6 patch releases (bug/security fixes). The dependency used a
minor-level pin (numpy==1.24.*) before PR apache#108 bumped it to 2.2.0.*, so
the patch-level pin appears unintentional.

Loosen to ==2.2.* to match the documented intent and allow 2.2.x patch
updates. Regenerate uv.lock (numpy 2.2.0 -> 2.2.6). Full test suite
(112 tests) and tox lint pass against numpy 2.2.6.

@Gerrrr Gerrrr 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.

Thanks @jordancharest !

@Gerrrr Gerrrr merged commit 8b78e9d into apache:master Jun 2, 2026
6 checks passed
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.

2 participants