-
-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
[Feature]: Does VLLM only support MistralModel Architecture for embedding? #7915
Comments
Yes, support for other models for embedding use needs to be added. |
@mgoin does it need implementation? or it has the same implementation with the current code? |
This issue has been automatically marked as stale because it has not had any activity within 90 days. It will be automatically closed if no further activity occurs within 30 days. Leave a comment if you feel this issue should remain open. Thank you! |
The support for embedding models has greatly improved now! https://docs.vllm.ai/en/latest/models/supported_models.html#text-embedding |
🚀 The feature, motivation and pitch
Does VLLM only support MistralModel Architecture for embedding?
I tried to force embedding_mode to true
model_config.embedding_mode = True,
this error raised:The text was updated successfully, but these errors were encountered: