-
Notifications
You must be signed in to change notification settings - Fork 4
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
Content download speed regression in Kolibri 0.16 #627
Comments
Joana reports that she has not been able to download a content pack once in the last three days. Bumping up priority to unbreak now. |
@dbnicholson @dylanmccall between you, can you figure out how we can work around this download speed regression for Ladybug? Particularly now that we download all channels' metadata during onboarding, and my reading of #549 is that the intention is to also eagerly download 500MB of thumbnails, this is a real issue. (I'm just a guy but, is it straightforward to revert the change that switched to chunked downloads? Or bump up the chunk size?) |
In #549 I'm just about done with background downloading and it worked fine in my testing. However, your point is completely valid. The most straightforward thing to do would be to copy the file transfer code from 0.16.0a12 (last release before the range requests were landed), and then wire up our own download manager and task in the explore plugin. It should be pretty mechanical. I updated my download test script to compare 0.15.12, 0.16.0a12 and 0.16.0a15 and 0.16.0a12 matched the 0.15.12 performance:
|
Perfect, has been merged and I kicked off a new release of the plugin. |
Ladybird 6.16-347 |
Verified Failed & It's a release blocker: It fails to proceed the download after you select a content pack. Test packages: Ladybird 6.16-347 & Windows 6.16.0.0 |
Yes, as Vanessa says, the application now shows a white screen right after Kolibri starts. In the logs I see a browser console error "get_download_status is not a function":
|
Oh but most important is this traceback error that happens before
|
A hotfix has been merged: #646 Deploying. |
Deployed:
|
Tracking upstream issue:
The text was updated successfully, but these errors were encountered: