Skip to content

Commit 2512f22

Browse files
sobychackospring-builds
authored andcommitted
Remove hardcoded topK in BedrockConverseProxyChatProperties (#3621)
Fixes: #3621 Bedrock does not support topK and setting it via BedrockConverseProxyChatProperties does not have any effect. Signed-off-by: Soby Chacko <[email protected]> (cherry picked from commit 8c411ab)
1 parent 2099278 commit 2512f22

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

auto-configurations/models/spring-ai-autoconfigure-model-bedrock-ai/src/main/java/org/springframework/ai/model/bedrock/converse/autoconfigure/BedrockConverseProxyChatProperties.java

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -33,11 +33,7 @@ public class BedrockConverseProxyChatProperties {
3333
public static final String CONFIG_PREFIX = "spring.ai.bedrock.converse.chat";
3434

3535
@NestedConfigurationProperty
36-
private ToolCallingChatOptions options = ToolCallingChatOptions.builder()
37-
.temperature(0.7)
38-
.maxTokens(300)
39-
.topK(10)
40-
.build();
36+
private ToolCallingChatOptions options = ToolCallingChatOptions.builder().temperature(0.7).maxTokens(300).build();
4137

4238
public ToolCallingChatOptions getOptions() {
4339
return this.options;

0 commit comments

Comments
 (0)