Skip to content
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

Bedrock document processing fixes #8005

Merged
merged 11 commits into from
Jan 29, 2025

Conversation

krrishdholakia
Copy link
Contributor

@krrishdholakia krrishdholakia commented Jan 25, 2025

  • refactor(factory.py): refactor async bedrock message transformation to use async get request for image url conversion

improve latency of bedrock call

  • test(test_bedrock_completion.py): add unit testing to ensure async image url get called for async bedrock call

  • refactor(factory.py): refactor bedrock translation to use BedrockImageProcessor

reduces duplicate code

  • fix(factory.py): fix bug not allowing pdf's to be processed

  • fix(factory.py): fix bedrock converse document understanding with image url

  • docs(bedrock.md): clarify all bedrock document types are supported

  • refactor: cleanup redundant test + unused imports

  • perf: improve perf with reusable clients

  • test: fix test

Copy link

vercel bot commented Jan 25, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
litellm ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 29, 2025 1:49am

@krrishdholakia krrishdholakia changed the title Litellm bedrock document processing improvements Bedrock document processing fixes Jan 26, 2025
@krrishdholakia krrishdholakia merged commit 8eaa5dc into main Jan 29, 2025
3 of 22 checks passed
@krrishdholakia krrishdholakia deleted the litellm_bedrock_document_processing_improvements branch January 29, 2025 01:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant