Skip to content

Conversation

@cztchoice
Copy link

@cztchoice cztchoice commented May 29, 2025

numpy 1 do not support python >= 3.13

and from link onnxruntime>=1.19.0 (released August 2024) is compatible with numpy>=2.0.0.

and we have already pin onnxruntime to 1.19.

So I think we can unpin numpy now

Related Issue

Checklist

  • I have read the CONTRIBUTING guidelines.
  • I have added tests to cover my changes.
  • I have updated the documentation (docs folder) accordingly.

Additional Notes

Add any other context about the PR here.

numpy 1 not support python >= 3.13

and from [link](microsoft/onnxruntime#21063 (comment))  onnxruntime>=1.19.0 (released August 2024) is compatible with numpy>=2.0.0.

and we have already pin onnxruntime to 1.19.

So I think we can unpin numpy now
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

Updates numpy dependency constraints in libs/infinity_emb/pyproject.toml to support Python 3.13 compatibility by allowing numpy >= 2.0.

  • Modified numpy version constraint from >=1.20.0,<2 to >=1.20.0 since onnxruntime >= 1.19.0 is compatible with numpy 2.0
  • Change is backwards compatible as the lower bound of >=1.20.0 remains unchanged

💡 (1/5) You can manually trigger the bot by mentioning @greptileai in a comment!

1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

@wirthual
Copy link
Collaborator

Nice, we should probably also extend the tests to cover 3.13 then

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