-
-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
[Platform] Do not raise error if _Backend is not found #12023
[Platform] Do not raise error if _Backend is not found #12023
Conversation
👋 Hi! Thank you for contributing to the vLLM project. 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 do one of these:
🚀 |
9e5a540
to
b411241
Compare
attn_backend = backend_name_to_enum(attn_backend.get_name()) | ||
if attn_backend in {_Backend.FLASH_ATTN, _Backend.FLASH_ATTN_VLLM_V1}: | ||
attn_backend = _Backend.XFORMERS | ||
backend = backend_name_to_enum(attn_backend.get_name()) |
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.
Rename here to make lint check happy
b411241
to
850bce0
Compare
9439f9e
to
edb061b
Compare
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.
LGTM now!
Please update the test to avoid the CI failure. |
edb061b
to
fd875db
Compare
fd875db
to
280ae90
Compare
Signed-off-by: wangxiyuan <[email protected]>
280ae90
to
065ed53
Compare
Signed-off-by: Mengqing Cao <[email protected]>
…12023) Signed-off-by: wangxiyuan <[email protected]> Signed-off-by: Mengqing Cao <[email protected]> Co-authored-by: Mengqing Cao <[email protected]> Signed-off-by: ice-tong <[email protected]>
…12023) Signed-off-by: wangxiyuan <[email protected]> Signed-off-by: Mengqing Cao <[email protected]> Co-authored-by: Mengqing Cao <[email protected]>
…12023) Signed-off-by: wangxiyuan <[email protected]> Signed-off-by: Mengqing Cao <[email protected]> Co-authored-by: Mengqing Cao <[email protected]>
…12023) Signed-off-by: wangxiyuan <[email protected]> Signed-off-by: Mengqing Cao <[email protected]> Co-authored-by: Mengqing Cao <[email protected]>
Part of #11162
The out-of-tree platform may have a different _Backend for attention backend. Ignore the error if the input _Backend is not the one in-tree.