griptape
griptape copied to clipboard
Add top_p, top_k parameters, add defaults to all fields
- [x] I have read and agree to the contributing guidelines for submitting new pull requests.
Describe your changes
- Added type hints to Bedrock clients.
- Reorganized Driver fields to place dependent fields (clients/tokenizers) after primitive fields.
- Added
top_p
field to all Prompt Drivers. - Added
BasePromptDriver.additional_params
to set additional request parameters such as top k. - Added defaults to all fields. This solves an issue where users would provide a
client
but still be required to pass in fields likeapi_key
. - Removed redundant
model
fields as it is provided by the base class.
Issue ticket number and link
Closes #236 Closes #293