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

add sleep to decrease cpu usage while waiting for response #51

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vikoff
Copy link

@vikoff vikoff commented Feb 16, 2015

before:
$ time webkit2png --xvfb 1024 1024 -g 1024 1024 --scale 200 200 --aspect-ratio=keep --timeout=30 --log='' -o /tmp/img.png http://long-site
real 0m10.553s
user 0m8.383s
sys 0m2.092s

after:
$ time webkit2png --xvfb 1024 1024 -g 1024 1024 --scale 200 200 --aspect-ratio=keep --timeout=30 --log='' -o /tmp/img.png http://long-site
real 0m11.074s
user 0m0.412s
sys 0m0.078s

@ghost
Copy link

ghost commented Jun 13, 2017

Isn't 0.5 seconds quite a lot?

#62 also has this change: 81460b1

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

Successfully merging this pull request may close these issues.

1 participant