deeplake icon indicating copy to clipboard operation
deeplake copied to clipboard

IPFS Storage Provider for decentralized storage

Open smejak opened this issue 1 year ago • 5 comments

🚀 🚀 Pull Request

Checklist:

  • [ ] My code follows the style guidelines of this project and the Contributing document
  • [ ] I have commented my code, particularly in hard-to-understand areas
  • [ ] I have kept the coverage-rate up
  • [ ] I have performed a self-review of my own code and resolved any problems
  • [ ] I have checked to ensure there aren't any other open Pull Requests for the same change
  • [ ] I have described and made corresponding changes to the relevant documentation
  • [ ] New and existing unit tests pass locally with my changes

Changes

Added the IPFSProvider to integrate decetralized storage (IPFS) into Activeloop Hub. Currently able to hub.load datasets stored on IPFS.

smejak avatar Jul 13 '22 13:07 smejak

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

:white_check_mark: smejak
:x: Jakub Smékal


Jakub Smékal seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Jul 13 '22 13:07 CLAassistant

Hey @smejak! Thanks for the PR, it is definitely a great feature. From a cursory glance, the code looks good, the team and I will review it properly in the next few days.

AbhinavTuli avatar Jul 14 '22 17:07 AbhinavTuli

Hey @AbhinavTuli! Thanks a lot! I am still working on the final implementation of copy and delete functions & their corresponding unit tests. I will push the updates to this PR once it's ready.

smejak avatar Jul 17 '22 18:07 smejak

HI @smejak, thanks for your contribution. Would you mind signing our CLA while we review your PR.

tatevikh avatar Jul 19 '22 09:07 tatevikh

Hi @tatevikh. I signed the CLA but then I accidentally pushed a commit with incorrect credentials. I tried to git rebase that commit but it seems it hasn't been updated here.

smejak avatar Jul 22 '22 06:07 smejak

Closing due to many conflicts. Will add the new PR link as soon as we open it.

tatevikh avatar Mar 31 '23 14:03 tatevikh