-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Adding model and config parameters to extract() #119
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
Conversation
Running live API tests... This will take a few minutes. |
✅ Live API tests passed! All endpoints are working correctly. |
Sorry about those styling issues. Incorporated your changes manually and left your PR untouched since it was targeting the the main branch on your repo rather than my fork. |
Thanks @mariano, really appreciate the PR! |
* Adding model and config parameters to extract() * Adding extract precedence tests * Applying changes from #120
* Adding model and config parameters to extract() * Adding extract precedence tests * Applying changes from google#120
* Adding model and config parameters to extract() * Adding extract precedence tests * Applying changes from #120
Description
Adds
config
andmodel
toextract()
while maintaining full backward compatibility.extract()
:config
: An optionalModelConfig
.model
: An optionalBaseModel
implementation.extract()
for model selection is as follows (fromchosen first to last):
model
config
model_id
(withapi_key
,temperature
andmodel_url
)language_model_type
(deprecated.)constraints
fromextract()
Fixes #106
How Has This Been Tested?
All tests succeed with:
Checklist:
pylint
over the affected code.