-
Notifications
You must be signed in to change notification settings - Fork 13.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
core[patch]: move some attr/methods to BaseLanguageModel #18936
core[patch]: move some attr/methods to BaseLanguageModel #18936
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
"""Metadata to add to the run trace.""" | ||
|
||
@validator("verbose", pre=True, always=True) | ||
def set_verbose(cls, verbose: Optional[bool]) -> bool: |
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.
was inconsistent before - this used to only be on llms but not chat model. Will now have same behavior for verbose=None
for both
verbose: bool = Field(default_factory=_get_verbosity) | ||
"""Whether to print out response text.""" | ||
callbacks: Callbacks = Field(default=None, exclude=True) | ||
"""Callbacks to add to the run trace.""" | ||
callback_manager: Optional[BaseCallbackManager] = Field(default=None, exclude=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.
left this in the two places because deprecated and didn't want to add deprecated attr to baselanguagemodel
…i#18936) Cleans up some shared code between `BaseLLM` and `BaseChatModel`. One functional difference to make it more consistent (see comment)
…i#18936) Cleans up some shared code between `BaseLLM` and `BaseChatModel`. One functional difference to make it more consistent (see comment)
Cleans up some shared code between `BaseLLM` and `BaseChatModel`. One functional difference to make it more consistent (see comment)
Cleans up some shared code between
BaseLLM
andBaseChatModel
. One functional difference to make it more consistent (see comment)