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

Memory issues in Edge #8

Open
jdahdah opened this issue Feb 22, 2017 · 1 comment
Open

Memory issues in Edge #8

jdahdah opened this issue Feb 22, 2017 · 1 comment

Comments

@jdahdah
Copy link

jdahdah commented Feb 22, 2017

Hey, I'm running into some memory issues with your script in MS Edge. I'm using your default maxWidth and maxHeight of 1024px. When I try to upload an image with the dimensions 7525×5019px, Edge 14.14393 on Windows 10 aborts the upload with the following error:

Not enough storage is available to complete this operation.

It points to line 167:
ctx.drawImage(img, 0, 0);

According to this, IE can handle a canvas of maximum 8,192×8,192px, and Edge can probably handle a lot more, so I should be well within the boundaries. I'm having no issues in IE10 or IE11. Any idea what could be causing this?

@rossturner
Copy link
Owner

Well outside of my area of expertise I'm afraid! If it wasn't working in any browser I'd guess the max texture size for your device didn't support 8096x8096 (roughly 90% of cards do as of not too long ago). As it is, it sounds like an issue with Edge and an admittedly unusual use of canvas elements. I'll do some googling on the subject as it sounds an interesting one, but I wrote this code several years before Edge even existed :)

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

2 participants