-
Notifications
You must be signed in to change notification settings - Fork 23
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
Download resources (models, datasets) on-demand #926
Comments
Does moving large files to some S3 buckets help? It depends on the cloud provider we use (note that S3 is not Amazon). Maybe it's the limitation on client side (limitation of Android Apps). @nathanw-mlc where is the right place to place big models (github, S3 buckets, or others). |
Files served to the app should be in S3, especially since our Cloudflare R2 buckets use Cloudflare's CDN to distribute data to users all around the world at high speeds. However, I also agree with Anh that the app should not download everything the moment you launch it. That is really strange behavior for an Android app. The first thing you expect to see when launching a new app is not the app downloading a whole bunch of data without even asking you first. What if you first launch the app while not connected to Wi-Fi and the app chews through your mobile data plan? Data should be downloaded on an as-needed-basis, and the app should also notify users that proceeding will require downloading X amount of data. |
@nathanw-mlc Can you please create a public bucket for hosting the models and datasets, maybe named |
@anhappdev I've created a public bucket named |
I uploaded a test file and tried with the URL: I didn't know you need to create a new domain for the bucket. I think we can also just create a new directory named |
My bad, I forgot to add
Yes, each subdomain corresponds to a single bucket. A new directory in the |
@nathanw-mlc Ok. I choose to use the |
We now have a lot of large files. The app should download the files only when it needs them and not at app launch.
The text was updated successfully, but these errors were encountered: