-
Notifications
You must be signed in to change notification settings - Fork 239
ci: allow to use recent numpy #826
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.
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
ci: allow to use recent numpy #826
Conversation
genai_perf package already includes this change triton-inference-server/perf_analyzer#275
@indrajit96 could you please kindly help to take a look? Thanks. |
@statiraju @krishung5 @kthui, would anyone be able to review this PR? I assume this should be fine for the client code. Edit: Just noticed that Yuan already tagged Indrajit. This will help unblock vLLM. Anyone's review/approval here would be appreciated. |
Hi @zhouyuan, have triggered a CI for this. |
CI looks good to me waiting for CI review and approval to merge |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CI looks good to me waiting for CI review and approval to merge
@zhouyuan this is merged |
@indrajit96 Thank you so much for the review and merge! Would it be convenient to included this commit in the coming thanks, -yuan |
This PR isn't yet included in the It'd be great to include this, it should be safe and unblocks a bunch of other projects (e.g., |
The Triton policy is generally not to patch prior releases. This commit was merged in between commits, so it will be included in our next monthly release, 25.04. CC: @statiraju |
This patch removed the "numpy<2" restrict.
genai_perf package already includes this change
triton-inference-server/perf_analyzer#275
Cc: @dyastremsky