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

Filecoin Integration #872

Open
wants to merge 59 commits into
base: main-old
Choose a base branch
from

Conversation

richardblythman
Copy link

Filecoin integration from core tech earmarks.
Complements PR to provider oceanprotocol/provider#513

Changes proposed in this PR:

  • Create StorageProvider class for uploading to decentralized storage
  • Integrate HuggingFace Hub library and create HuggingFaceHub class for uploading to data science platforms
  • Modify encrypt function to encrypt file content (as well as urls and ddos)
  • Create an environmental variable for Filecoin urls
  • Add class for Filecoin file structures
  • Readme tutorial flow for uploading to storage and HuggingFace
  • Add tests for uploading to and downloading from decentralized storage and HuggingFace

@AnaLoznianu
Copy link
Member

Hi @richardblythman! Thanks for opening this PR! Unfortunately, we cannot review the ocean.py work until the provider work is done and merged. You know, components that depend on other components. Can you please update this one first and mark it as ready for review when you are done?
Thank you 🙏

@richardblythman
Copy link
Author

Hey @AnaLoznianu. We've been waiting for quite some time to hear from Ocean on merging these PRs. We'll probably need to get them to up to date with the newer versions of provider and ocean.py. If we put in the time to do that, are you ready to work with us to merge them?

@AnaLoznianu
Copy link
Member

Hi @richardblythman :) From a processes perspective, firstly we need to integrate the changes in the provider. As the provider PR is still in draft mode we assumed it is not ready for review.
For a technical review, I'll defer this to @alexcos20

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.

None yet

3 participants