Add temperature parameter to process_image#21
Merged
Conversation
Adds an optional `temperature:` keyword argument (default 0.0) to `Client#process_image`, forwarded to the Bedrock converse API via `inference_config`. Includes tests for default and custom temperature values. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…b to comply with rubocop
Collaborator
|
Karthik Koralla (@karthikreddy-02) Were you able to test this against any of the LLMs? Do you have the access keys to do that? I would like to confirm that the LLMs we have registered are able to take these inputs. Also, I don't think we should default to I can get you keys if you don't have them. |
Contributor
Author
|
Alex Kiessling (@ajkiessl) I don't have the access keys. I believe we can test outputs for different temperatures and see what fits best and change the default temperature based on that. |
Collaborator
|
Karthik Koralla (@karthikreddy-02) That sounds like a good idea. I can get you the keys tomorrow. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Adds an optional
temperature:keyword argument (default 0.0) toClient#process_image, forwarded to the Bedrock converse API viainference_config. Includes tests for default and custom temperature values.Closes #18