-
Notifications
You must be signed in to change notification settings - Fork 31
[s390x] Update s390x depencies #494
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
Conversation
Signed-off-by: Nikhil-Suryawanshi <[email protected]>
|
👋 Hi! Thank you for contributing to vLLM support on Spyre. Or this can be done with Now you are good to go 🚀 |
pyproject.toml
Outdated
| # Skip packages only on s390x, expected to be pre-installed | ||
| "vllm ; platform_machine != 's390x'", | ||
| "ray; platform_machine != 's390x'", | ||
| "ray[default]; platform_machine != 's390x'", |
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.
I think this is causing all the ray[default] deps to be pulled in for x86 (see all the uv.lock additions) which is not what we want
Signed-off-by: Nikhil-Suryawanshi <[email protected]>
| ] | ||
|
|
||
| [[package]] | ||
| name = "cupy-cuda12x" |
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.
nice!
Not sure how this was pulled in anyway- good to see it removed. Maybe this was from ray[cgraph]?
joerunde
left a comment
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.
nice
# Description This PR follows [494](#494). Certain dependencies of vllm-spyre are not readily available on ppc64le arch. These will be skipped and handled manually. Signed-off-by: Daniel Schenker <[email protected]>
Description
Few pakcages are not available for s390x and usually built by source, skipping those packages from pyproject and executed
uv lock