Skip to content

Remove numpy version pin - #2788

Merged
Xiaoyu Z (xiaoyu-work) merged 1 commit into
mainfrom
xiaoyu/numpy
Nov 5, 2026
Merged

Xiaoyu Z (xiaoyu-work) merged 1 commit into
mainfrom
xiaoyu/numpy

Conversation

@xiaoyu-work

Copy link
Copy Markdown
Member

Describe your changes

Checklist before requesting a review

  • Add unit tests for this change.
  • Make sure all tests can pass.
  • Update documents if necessary.
  • Lint and apply fixes to your code by running lintrunner -a
  • Is this a user-facing change? If yes, give a description of this change to be included in the release notes.

(Optional) Issue link

Copilot AI balanced review requested due to automatic review settings November 4, 2026 23:05

Copilot AI 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.

🟢 Approval recommended

No active review issues remain for the change removing the NumPy upper bound from test dependencies.

No open issues 🎉

What changed in this PR

Removes the NumPy version cap from Olive’s shared test requirements, allowing newer versions subject to other dependency constraints.

Changes:

  • Replaces numpy<2.0.0 with unversioned numpy.
File Description
test/​requirements-test.txt Removes the NumPy upper bound from test dependencies.

Review effort: Balanced

@xiaoyu-work
Xiaoyu Z (xiaoyu-work) merged commit fd466c4 into main Nov 5, 2026
13 checks passed
@xiaoyu-work
Xiaoyu Z (xiaoyu-work) deleted the xiaoyu/numpy branch November 5, 2026 00:07
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