-
Notifications
You must be signed in to change notification settings - Fork 16.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: Add solar model chat model #18556
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
temperature = 0.3 | ||
|
||
class Config: | ||
allow_population_by_field_name = True |
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.
what is this for? any comments?
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.
It's a config for pydantic object. The detailed explanation lies here. https://docs.pydantic.dev/1.10/usage/model_config. In this pr, we just follow the common sense in this repo such as openai implementation.
allow_population_by_field_name = True |
return values | ||
|
||
@root_validator() | ||
def validate_environment(cls, values: Dict) -> Dict: |
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.
I assumed this function would raise an exception if it detected any issues in its environment. Did I overlook something?
Example: | ||
.. code-block:: python | ||
from langchain_community.chat_models.solar import SolarChat | ||
solar = SolarChat(model="solar-v1-8k") |
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.
Should we use SolarLLM for greater generality?
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.
If we could support normal api calls, I believe we could let users touch any model within our solar system.
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.
Love the pr. Let's just ship it.
@efriis Just found the vercel deployment is failed. Do you have any suggestion? |
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.
Hey there! The current integration is failing some of our standard lint checks. Here's the guide on running and fixing them! https://python.langchain.com/docs/contributing/code
I can get around to fixing some of them this week if it's difficult to set up.
@efriis @baskaryan Thanks for your help again. The pr is blocked by this message: |
Add our solar chat models, available model choices: * solar-1-mini-chat * solar-1-mini-translate-enko * solar-1-mini-translate-koen More documents and pricing can be found at https://console.upstage.ai/services/solar. The references to our solar model can be found at * https://arxiv.org/abs/2402.17032 --------- Co-authored-by: Bagatur <[email protected]> Co-authored-by: Bagatur <[email protected]>
Add our solar chat models, available model choices: * solar-1-mini-chat * solar-1-mini-translate-enko * solar-1-mini-translate-koen More documents and pricing can be found at https://console.upstage.ai/services/solar. The references to our solar model can be found at * https://arxiv.org/abs/2402.17032 --------- Co-authored-by: Bagatur <[email protected]> Co-authored-by: Bagatur <[email protected]>
Add our solar chat models, available model choices:
More documents and pricing can be found at https://console.upstage.ai/services/solar.
The references to our solar model can be found at