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

Disproportionately high RAM use #1

Open
dgw opened this issue Aug 20, 2017 · 1 comment
Open

Disproportionately high RAM use #1

dgw opened this issue Aug 20, 2017 · 1 comment

Comments

@dgw
Copy link
Contributor

dgw commented Aug 20, 2017

Just found this tonight and, wow, it does exactly what it says! Seems pretty magical, frankly.

The only thing that tempered my sense of awe was seeing the browser process hosting this webapp shoot up to almost 2GB of RAM in task manager while preparing a 42MB ZIP file of images. Chrome itself became unresponsive for a couple minutes while Windows frantically paged memory to disk to let the app finish its work. (I have an embarrassingly small amount of RAM for 2017.)

This happened twice, with two different albums (one 42MB, one 29MB).

Given Stuk/jszip#135, I'd wager that the RAM use is coming from the ZIP file creation itself. I realize this app is probably using a different library, but JavaScript-based ZIP is still JavaScript-based ZIP, and is likely to still face similar issues.

Probably-important info about my environment:

  • Chrome 60.0.3112.101 (64-bit)
  • Windows 7 x64
  • 4GB physical memory (the reason I usually have task manager open: to see what's hogging RAM)
@dschep
Copy link
Owner

dschep commented Aug 20, 2017

Oof. That really is a lot of ram. That is the js library I'm using, but since i built this 4 years ago it's using v1. I could maybe upgrade that library if newer versions have better memory performance.

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