-
-
Notifications
You must be signed in to change notification settings - Fork 7.7k
[Model]: Add transformers
backend support
#11330
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
Co-authored-by: Isotr0py <[email protected]>
👋 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:
🚀 |
Hello @ArthurZucker! This is very exciting! I know this PR is still a draft, but could you provide some context on the scope of this effort? Is it to support any model on |
Yep, overall this should support We are refactor our models to make sure it's propagated to as many models as possible! |
Might not have time to finish this week, will make it ready for next week 🎄 |
Signed-off-by: Harry Mellor <[email protected]>
Signed-off-by: Harry Mellor <[email protected]>
Signed-off-by: Harry Mellor <[email protected]>
Signed-off-by: Harry Mellor <[email protected]>
Signed-off-by: Harry Mellor <[email protected]>
Signed-off-by: Harry Mellor <[email protected]>
…orted Signed-off-by: Harry Mellor <[email protected]>
Signed-off-by: Harry Mellor <[email protected]>
Signed-off-by: Harry Mellor <[email protected]>
Signed-off-by: Harry Mellor <[email protected]>
Signed-off-by: Harry Mellor <[email protected]>
Signed-off-by: Harry Mellor <[email protected]>
Benchmarks on A100 using the following command: python benchmarks/benchmark_throughput.py --backend vllm --model meta-llama/Llama-3.1-8B-Instruct --dataset ShareGPT_V3_unfiltered_cleaned_split.json Results:
|
Signed-off-by: Harry Mellor <[email protected]>
|
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.
Let's get this merged first! We can add BNB and LoRA support in other following PR.
Signed-off-by: Isotr0py <[email protected]>
Please fix the failing tests |
Signed-off-by: Isotr0py <[email protected]>
Signed-off-by: Isotr0py <[email protected]>
Please also add the distributed transformers test to the distributed tests CI |
Signed-off-by: Isotr0py <[email protected]>
Co-authored-by: Harry Mellor <[email protected]>
Signed-off-by: Isotr0py <[email protected]>
# Adds support for `transformers` as a backend Following huggingface/transformers#35235, a bunch of models should already be supported, we are ramping up support for more models. Thanks @Isotr0py for the TP support, and @hmellor for his help as well! This includes: - `trust_remote_code=True` support: any model on the hub, if it implements attention the correct way can be natively supported!! - tensor parallel support --------- Signed-off-by: Harry Mellor <[email protected]> Signed-off-by: Isotr0py <[email protected]> Co-authored-by: Isotr0py <[email protected]> Co-authored-by: Harry Mellor <[email protected]> Co-authored-by: Isotr0py <[email protected]> Co-authored-by: Cyrus Leung <[email protected]> Co-authored-by: Michael Goin <[email protected]> Co-authored-by: Isotr0py <[email protected]>
Adds support for
transformers
as a backendFollowing huggingface/transformers#35235, a bunch of models should already be supported, we are ramping up support for more models.
Thanks @Isotr0py for the TP support, and @hmellor for his help as well!
This includes:
trust_remote_code=True
support: any model on the hub, if it implements attention the correct way can be natively supported!!