-
Notifications
You must be signed in to change notification settings - Fork 42
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
Add api args and verbose parameters to Bedrock provider #292
base: main
Are you sure you want to change the base?
Add api args and verbose parameters to Bedrock provider #292
Conversation
…eter for debugging requests/responses.
… and more context. Minor comment changes
It's definitely an oversight that |
@billsanto thanks for working on this! As @atheriel says, I think this is probably a bit big, and anticipates some changes (like #280) that we'd rather tackle globally. Would you mind refocussing this PR on just enabling |
Thanks--I figured the verbose option was more appropriate as a central feature, so yes I can refocus on the api_args for the PR. I really needed that functionality to figure out what was going on under the hood. Is there a way I get both headers and payloads using httr2 directly? This is what the verbose output currently looks like:
(Some manual redactions made for the post). |
You can wrap any call with |
Added api_args and verbose parameters to the Bedrock provider. api_args is a list for parameters such as temperature, top_p, top_k, max_tokens, and stop_sequences. Added validation for each of these parameters. The verbose parameter adds useful information for developers and end users to inspect request and response headers and payloads. Updated the Roxygen documentation.