-
Notifications
You must be signed in to change notification settings - Fork 14.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
community: Add Naver chat model & embeddings #25047
community: Add Naver chat model & embeddings #25047
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
5b63962
to
79919e3
Compare
howdy! could you reopen this from a personal repo instead of an organization one? https://python.langchain.com/v0.2/docs/contributing/faq/#how-do-i-allow-maintainers-to-edit-my-pr this helps us collaborate on the code in your PR :) one question on namespaces - would it make sense to have these in also, could you clarify the difference between |
another PR(#25162) is reopen.
I would like to keep it as is, unless it is essential to unify the package name.
There are 3 models that are being serviced by embedding, and all are supported in the current PR. In addition, all the functionality of CLOVA Studio that serves actual models, such as distinguishing between test apps and service apps, is supported. The existing PR does not support this content because it is hard-coded. |
Description
Twitter handle: