João Galego
João Galego
When will this fix be merged?
All checks have passed and this fix works locally. Why is this still open?
Any news?
@beggers reaching out to you b/c you were the last one to review+approve+merge changes to `utils/embedding_functions.py` (https://github.com/chroma-core/chroma/pull/1517). Can we find a reviewer for this one?
> Please also update the embedding model documentation: https://github.com/chroma-core/docs/blob/main/docs/embeddings.md > > Currently the `AmazonBedrockEmbeddingFunction` is missing from the docs entirely. There's an open PR from another contributor for the docs...
@tazarov any other changes?
Hey @thinkwee, reaching out b/c you're currently listed as the top contributor. Can we have someone review this PR?
@usamimeri @Wei-Jianan as authors of [Bedrock-related PRs](https://github.com/geekan/MetaGPT/pulls?q=is%3Apr+bedrock), would you mind reviewing these changes? cc: @geekan
@karthink @csheaff Found this issue by accident. Maybe this will be useful for a future implementation >>> [`cl-bedrock`](https://github.com/JGalego/cl-bedrock)? Includes support for `InvokeModel`, `Converse` and `ApplyGuardrail` APIs. 