-
Notifications
You must be signed in to change notification settings - Fork 282
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
Speed Up Preloading #69
Conversation
have you tested it? |
iirc the reason I didn't have it be async is that it literally crashed and overloaded the filesystem cache |
yes i did |
it literally worked fine |
i can chunk it if that works better on lower end devices |
What browser and version? Did you fully clear site data? |
breaks parts of the cache for me |
half works for me, temporarily degraded filesystem but after like 20s it started working normally, I'm on a Samsung 980 nonpro NVME ssd, and i7-8700 with 48GB ram |
to be more specific, it's dropping files, if you look at the netlog some don't load properly when offline |
yup thats usually what I do too |
I have a feeling this issue is heavily dependent on system performance, since it relies on how fast it can store the files into the filesystem |
but the filesystem is definitely still being overloaded here, even on my machine and I suspect on CE's its passing the limit on how many files could have been overloaded |
does anyone here have a Google Octopus board chromebook to test on? |
i will check, im also implementing chunking rn |
@FoxMoss, system specifications? |
i do have a octopus chromebook but i also do not have time until tomorrow. |
I also have hardware acceleration turned off on chrome |
Fuckin git |
Added chunking though 👍 |
yeah i know |
i literally signed all the commits idk what its on about, i might need to make a new pr |
you added your PGP key on github and added your signing key to git? |
yeah it doesn't say [verified] next to your commits so i don't think you added it |
Signed-off-by: FoxMoss <[email protected]>
Signed-off-by: FoxMoss <[email protected]>
Signed-off-by: FoxMoss <[email protected]>
@CoolElectronics test? |
things are still broken, just not because of this commit |
Tiny patch, it just loads preloads file async now.