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.

Sign up for GitHub

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

Jump to bottom

community: bytes as a source to AzureAIDocumentIntelligenceLoader #26618

Merged
merged 3 commits into from
Nov 7, 2024
Merged

community: bytes as a source to AzureAIDocumentIntelligenceLoader #26618

merged 3 commits into from
Nov 7, 2024

Conversation

akseljoonas
Copy link
Contributor

  • Description: This PR adds functionality to pass in in-memory bytes as a source to AzureAIDocumentIntelligenceLoader.
  • Issue: I needed the functionality, so I added it.
  • Dependencies: NA
  • Twitter handle: @akseljoonas if this is a big enough change :)

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Sep 18, 2024
@vercel Vercel
Copy link

vercel bot commented Sep 18, 2024 •
edited
Loading

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

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Visit Preview Nov 7, 2024 3:38am

@dosubot dosubot bot added community Related to langchain-community Ɑ: doc loader Related to document loader module (not documentation) labels Sep 18, 2024
@akseljoonas
Copy link
Contributor Author

@efriis

@adamkovari
Copy link

+1 :))

@langcarl
Copy link

langcarl bot commented Nov 4, 2024

Thank you for the PR. This PR is marked Needs Support and has not yet received the 5 upvotes required by maintainers for review. It has been open for at least 25 days. Per the LangChain review process, this PR will be closed in 5 days if it does not reach the required threshold.

The Needs Support status is intended to prioritize review time on features that have demonstrated community support. If you feel this status was assigned in error or need more time to gather the required upvotes, please ping (at)ccurme and (at)efriis.

@langcarl langcarl bot added the stale Issue has not had recent activity or appears to be solved. Stale issues will be automatically closed label Nov 4, 2024
@MikaMann
Copy link

MikaMann commented Nov 6, 2024

Yes!

@dosubot dosubot bot added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label Nov 7, 2024
@efriis efriis self-assigned this Nov 7, 2024
@efriis efriis enabled auto-merge (squash) November 7, 2024 03:38
@efriis efriis merged commit 2cb3927 into langchain-ai:master Nov 7, 2024
18 checks passed
@LukaszSawala
Copy link

YES!!!

yanomaly pushed a commit to yanomaly/langchain that referenced this pull request Nov 8, 2024
@akseljoonas @efriis @yanomaly
community: bytes as a source to AzureAIDocumentIntelligenceLoader (l…
5838d0f 
…angchain-ai#26618)

- **Description:** This PR adds functionality to pass in in-memory bytes
as a source to `AzureAIDocumentIntelligenceLoader`.
- **Issue:** I needed the functionality, so I added it.
- **Dependencies:** NA
- **Twitter handle:** @akseljoonas if this is a big enough change :)

---------

Co-authored-by: Aksel Joonas Reedi <aksel@klippa.com>
Co-authored-by: Erick Friis <erick@langchain.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@efriis efriis efriis approved these changes

Assignees

@efriis efriis

Labels
community Related to langchain-community Ɑ: doc loader Related to document loader module (not documentation) lgtm PR looks good. Use to confirm that a PR is ready for merging. size:M This PR changes 30-99 lines, ignoring generated files. stale Issue has not had recent activity or appears to be solved. Stale issues will be automatically closed
Projects
PR Reviews
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@akseljoonas @adamkovari @MikaMann @LukaszSawala @efriis