[Misc] Remove misleading log#13266
Conversation
Signed-off-by: wangxiyuan <wangxiyuan1007@gmail.com>
|
👋 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 🚀 |
|
@maxdebayser @khluu Is this change fine? Thanks. |
|
@wangxiyuan , since this is a local path, I think that vllm shouldn't even enter this retry loop, it should search for the files locally first. Let me try to reproduce this behavior here. |
|
@wangxiyuan , I've opened PR #13292 to fix this behavior. Thanks for reporting the problem! |
|
see #13292 |
When loading a local model like:
The log will show some info in error level, but actually it doesn't raises any error.
This PR remove this misleading log.
Related to: #13107