-
-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
[Usage]: vllm: error: unrecognized arguments: --lora-path #13669
Comments
According to the docs, you should use |
@imrankh46 - like @DarkLight1337 mentioned, try replacing |
@varun-sundar-rabindranath showing the following error |
@varun-sundar-rabindranath how did you serve lora adapter? |
oh apologize. vllm not supporting modules_to_save. can you add support for it on the fly...
|
@imrankh46 - I'll take a look. cc'ing @DarkLight1337 @jeejeelee . |
Hi @imrankh46 . I am unfamiliar with As far as I understand, Please correct me if the suggestion is non-sensical. Also, I believe you are using a different LoRA adapter (not yard1/llama-2-7b-sql-lora-test), can you share some commands so I can repro and try things for myself. Thanks. |
@varun-sundar-rabindranath I am using a different adapter. When we add different tokens to the tokenizer, such as |
@varun-sundar-rabindranath also suggest by trl team read the following |
I see. I think I understand the issue a bit better now.
Line 138 in 3e472d8
Line 902 in 3e472d8
@jeejeelee @DarkLight1337 , looks like we could potentially handle this at the |
See: #11714 |
Your current environment
How would you like to use vllm
i am using vllm
version 0.7.3
and it have already support for lora but it's showing the following error.it showing the following error.
vllm: error: unrecognized arguments: --lora-path yard1/llama-2-7b-sql-lora-test
Before submitting a new issue...
The text was updated successfully, but these errors were encountered: