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

Long delay between steps 1 and 2 on mothertongues in French #116

Closed
joanise opened this issue Feb 16, 2023 · 8 comments
Closed

Long delay between steps 1 and 2 on mothertongues in French #116

joanise opened this issue Feb 16, 2023 · 8 comments

Comments

@joanise
Copy link
Member

joanise commented Feb 16, 2023

When I use
https://readalong-studio.mothertongues.org/fr/
and create a read along, the first time, it can take 5 to 10 seconds with no user feedback.

The ss progress bar is visible for about a second, so alignment is presumably complete, but I guess it has to load the web-component js files or something like that, and in that case there is no user feedback, so it just looks like it's frozen, when in fact I assume it's loading stuff.

It would be nice for the progress bar, or something, to keep moving in one way or another while we're waiting for stuff to load.

@joanise
Copy link
Member Author

joanise commented Feb 16, 2023

With cache disabled, it looks like loading all those polyfills links is the biggest chunks:

image

I know it doesn't look that slow here, but when I first ran it fresh, it felt quite a bit longer.

And now I just reran it with the cache enabled, and this was egregious:

image

But third time's the charm, it only took 223ms when I did it a second time with the cache enabled.

So... it's kind of just a first time only problem, but I'd still like some spinner or visible thing while we're loading stuff, so the user knows there's something happening. I've had several situations in the last few days where I thought it was just broken, went away to do something else in another window, and then oops it appeared. We probably can't avoid that, but we should make it so the user knows it's happening.

@joanise
Copy link
Member Author

joanise commented Feb 16, 2023

This is funny, though:
image
even when things were fast, it's still spending 60ms confirming the cache is still up to date for those two 302's...

@joanise
Copy link
Member Author

joanise commented Feb 16, 2023

As for the 39s + 24s 302's above, I wonder if it might have been just after a redeploy on the server side, since we've been actively pushing to main.

@dhdaines
Copy link
Collaborator

As for the 39s + 24s 302's above, I wonder if it might have been just after a redeploy on the server side, since we've been actively pushing to main.

yeah, probably that! sorry!

@joanise
Copy link
Member Author

joanise commented Feb 16, 2023

No apologies required, it was good to observe it.

@dhdaines
Copy link
Collaborator

The polyfills chunks are actually the model files for soundswallower, which are several megabytes in size and can't really be compressed - unfortunately at the moment we can't provide exact progress whlie loading them, but we can perhaps change the progress bar to do a wavy thing while we call decoder.initialize(). I had thought it was doing this but apparently not, I can look at it.

@dhdaines
Copy link
Collaborator

This is probably fixed by #122

@roedoejet
Copy link
Collaborator

Fixed by #122 until proven otherwise :)

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

3 participants