langchain icon indicating copy to clipboard operation
langchain copied to clipboard

community: Add Support for Loading Documents from Alibaba Cloud OSS file.

Open pcliupc opened this issue 9 months ago • 2 comments

  • Description: Add document loader for loading file from alibaba cloud OSS (object storage service)
  • Issue: None
  • Dependencies: Install oss2 python package.
  • Twitter handle: None

pcliupc avatar May 11 '24 12:05 pcliupc

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

Name Status Preview Comments Updated (UTC)
langchain ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 21, 2024 7:17am

vercel[bot] avatar May 11 '24 12:05 vercel[bot]

@baskaryan Please help review this pr.

pcliupc avatar May 16 '24 13:05 pcliupc

I'd suggest following this PR: https://github.com/langchain-ai/langchain/pull/21957

  1. We generally just need a primitive which is a BaseBlobLoader that only yields Blobs, without any parsing logic at all.
  2. Could you remove any changes in langchain package. Changes should be strictly in langchain-community

eyurtsev avatar May 21 '24 18:05 eyurtsev

I'd suggest following this PR: #21957

  1. We generally just need a primitive which is a BaseBlobLoader that only yields Blobs, without any parsing logic at all.
  2. Could you remove any changes in langchain package. Changes should be strictly in langchain-community

sure, I will keep an eye on this PR.

pcliupc avatar May 23 '24 07:05 pcliupc

I'd suggest following this PR: #21957

  1. We generally just need a primitive which is a BaseBlobLoader that only yields Blobs, without any parsing logic at all.
  2. Could you remove any changes in langchain package. Changes should be strictly in langchain-community @eyurtsev The CloudBlobLoader implemented in this PR #21957 supports three kinds of cloud path, are you suggesting extending CloudBlobLoader to support more? And BTW, CloudBlobLoader relies on CloudPathLib library which also only supports three kinds of cloud path.

pcliupc avatar May 30 '24 04:05 pcliupc

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[bot] avatar Nov 04 '24 19:11 langcarl[bot]