-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
community[patch]: Fix typo in modelName options of BaiduQianfanEmbeddings
#5970
community[patch]: Fix typo in modelName options of BaiduQianfanEmbeddings
#5970
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Thank you! |
This class was actually deprecated in favor of this one: Could you change it there too? |
CC @dl102306 |
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.
thanks for catching this!
BaiduQianfanEmbeddings
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.
Missed Jacob's comments. Requesting changes until resolved
@dl102306 could you verify this is good? |
Thank you! |
model name
bge-large-en
should bebge_large_en
likebge_large_zh
at line 188 in the same file:
https://aip.baidubce.com/rpc/2.0/ai_custom/v1/wenxinworkshop/embeddings/${this.modelName}?access_token=${this.accessToken}
${this.modelName}
directly used in the URLURL for the image:
https://cloud.baidu.com/doc/WENXINWORKSHOP/s/mllz05nzk