-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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] Want to get the last_hidden_states
, is there an interface for that? If not, what code should be modified to realize it?
#853
Comments
我也很需要这个API,来提取embedding,forward处开始改的话太痛苦了, |
@WoosukKwon @simon-mo @zhuohan123 is this a feature that you'd like to see implemented? |
https://github.com/WuNein/vllm4mteb/blob/main/vllm-new.py |
@WuNein It looks great! Will you create a PR to the main vllm so that we may using vllm to serving embedding model? |
Its more useful if we can support decode-based embedding model with v1/embedding api like openai embedding api. |
How would I say, someone do something |
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! |
This issue has been automatically closed due to inactivity. Please feel free to reopen if you feel it is still relevant. Thank you! |
I looked into the source code and found that the class
Sampler
discards the prefix inlast_hidden_states
.Is it possible for me to start with
Sampler
and implement the output tolast_hidden_states
as an optional output?Could the development team or anyone else familiar with vLLM provide some guidance and suggestions?
想要获取 last_hidden_states,有无对应接口?如果没有,应该修改哪些代码来实现?
我仔细查看了源代码,发现类
Sampler
舍弃了last_hidden_states
中的前缀。我是否可以从
Sampler
开始修改,以可选输出的形式实现对last_hidden_states
的输出?请问开发团队或其他任何熟悉 vLLM 的人能否提供一些指导和建议?
The text was updated successfully, but these errors were encountered: