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

RAM usage #32

Closed
zetok opened this issue Mar 18, 2016 · 1 comment
Closed

RAM usage #32

zetok opened this issue Mar 18, 2016 · 1 comment
Labels
I-High Issues that are breaking core functionality for a small number of users

Comments

@zetok
Copy link

zetok commented Mar 18, 2016

Oxipng aec8e41

dat ram

Used image: https://gist.github.com/zetok/1452501c644959325d45

pls fix?

@shssoichiro
Copy link
Owner

You're right, it looks like this is an issue when using high optimization levels (-o 6). oxipng is trying to spawn all the compression threads at once, and each thread uses memory. This isn't a problem with -o 2 (8 threads), but -o 6 spawns 648 threads at once.

I'll need to add a limit to the number of threads spawned at one time. Probably 1.5x the number of CPU cores.

@shssoichiro shssoichiro added the I-High Issues that are breaking core functionality for a small number of users label Mar 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I-High Issues that are breaking core functionality for a small number of users
Projects
None yet
Development

No branches or pull requests

2 participants