force quit when get pynvml version issue#12972
force quit when get pynvml version issue#12972lucypham951 wants to merge 1 commit intovllm-project:mainfrom
Conversation
|
👋 Hi! Thank you for contributing to the vLLM project. 💬 Join our developer Slack at https://slack.vllm.ai to discuss your PR in #pr-reviews, coordinate on features in #feat- channels, or join special interest groups in #sig- channels. Just a reminder: PRs would not trigger full CI run by default. Instead, it would only run Once the PR is approved and ready to go, your PR reviewer(s) can run CI to test the changes comprehensively before merging. To run CI, PR reviewers can either: Add 🚀 |
|
The issue should be solved by #12963. cc: @youkaichao |
|
Closing as superseded by #12963 |
This PR fixes an issue where CUDA detection fails silently when an outdated
version of
pynvml(<12.0) is installed. The previous implementationswallowed the
RuntimeError, leading tois_cuda=Falseeven when CUDAdevices were available, leading to UnspecifiedPlatform --> NotImplementedError
terminate when found version error.