-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
show error message when loading files that are too big #2981
Comments
Taking this up as part of GSoC |
hi @TanviKumar what's the status on this one? |
Hey Lauren, the limit for loading large files was found as 64MB on previous
versions of Firefox, so the goal now is to identify when to show an error
stating that the file size is too large.
Also planning to add this to the documentation of the loading methods so
the users are aware.
|
Hi @kjhollen, in order to limit the user from the loading very large files (in this case anything greater than 64MB in size), I plan on using the HEAD request to obtain the content length of the file without fetching the entire body. Would it be wise to use synchronous XHR or Async XHR? Here's a link to what I've been referring to. |
I think async will be better just so that if the server somehow has long latency it will not block the whole js thread causing the user's sketch to stop. |
Nature of issue?
Most appropriate sub-area of p5.js?
Feature enhancement details:
There are currently two open issues for when the browser fails to load a file because it is too big: #2661 for
loadStrings()
and #2216 forloadTable()
. I think they are not possible to fix, but it would be good to produce an error message for the user and suggest breaking the data into smaller files as a work around.New feature details:
The text was updated successfully, but these errors were encountered: