docs: docsting for call()
in BaseGoogle
LLM
#577
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
An agrument called
value
seems to be deprecated in current version for Google LLMs classes (it was added in baedcbc). Remove the docsting.Update docstrings for
instruction
(too general typeobject
was specified, this mismatches type hints).Update docstrings for
suffix
.value
argumentinstruction
suffix
description to look more verboseSummary by CodeRabbit
call
method in the base class of thepandasai/llm/base.py
file. The changes include removal of thevalue
parameter and modification of thesuffix
parameter's default value. Additionally, theinstruction
parameter has been annotated with the typePrompt
. These changes streamline the function signature and enhance code readability without altering the core functionality.