Browse Source

Fix/bedrock issues (#2718)

### What problem does this PR solve?

Adding a Bedrock API key for Claude Sonnet was broken. I find the issue
came up when trying to test the LLM configuration, the system is a
required parameter in boto3.

As well, there were problems in Bedrock implementation for embeddings
when trying to encode queries.

### Type of change

- [X] Bug Fix (non-breaking change which fixes an issue)
tags/v0.13.0
Omar Leonardo Sanchez Granados 1 year ago
parent
commit
34761fa4ca
No account linked to committer's email address
2 changed files with 4 additions and 3 deletions
  1. 3
    2
      rag/llm/chat_model.py
  2. 1
    1
      rag/llm/embedding_model.py

+ 3
- 2
rag/llm/chat_model.py View File

@@ -630,7 +630,7 @@ class BedrockChat(Base):
modelId=self.model_name,
messages=history,
inferenceConfig=gen_conf,
system=[{"text": system}] if system else None,
system=[{"text": (system if system else "Answer the user's message.")}] ,
)
# Extract and print the response text.
@@ -675,7 +675,8 @@ class BedrockChat(Base):
streaming_response = self.client.converse_stream(
modelId=self.model_name,
messages=history,
inferenceConfig=gen_conf
inferenceConfig=gen_conf,
system=[{"text": system if system else ""}],
)

# Extract and print the streamed response text in real-time.

+ 1
- 1
rag/llm/embedding_model.py View File

@@ -443,7 +443,7 @@ class BedrockEmbed(Base):

response = self.client.invoke_model(modelId=self.model_name, body=json.dumps(body))
model_response = json.loads(response["body"].read())
embeddings.extend([model_response["embedding"]])
embeddings.extend(model_response["embedding"])

return np.array(embeddings), token_count


Loading…
Cancel
Save