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

Optional dependencies for GCS/S3/HuggingFace-Hub? #223

Open
sidharthrajaram opened this issue Mar 11, 2024 · 1 comment
Open

Optional dependencies for GCS/S3/HuggingFace-Hub? #223

sidharthrajaram opened this issue Mar 11, 2024 · 1 comment

Comments

@sidharthrajaram
Copy link

Hi, big fan of the project!

cached_path is certainly very capable because of the support for GCS/S3/HuggingFace out of the box, but one issue is that it seems to bring a lot of dependency baggage for situations that really only require HTTP/local path functionality.

For example, one project I am working with relies on the HTTP and local cache path functionality, but there are conflicts due to the boto3 package. It's kind of like a printer refusing to print black-and-white when there is no magenta ink.

To that end, I'm curious whether there's interest in making the HF/S3/GCS dependencies optional? I can make a PR if needed! Thank you!

@epwalsh
Copy link
Member

epwalsh commented Mar 11, 2024

Hey @sidharthrajaram yes we'd be happy to accept a PR for this.

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

No branches or pull requests

2 participants