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

Speed up thumbnail generation for PDFs #399

Merged

Conversation

Projects
None yet
2 participants
@jat255
Copy link
Contributor

commented Sep 5, 2018

Since we only use the first page of the PDF for thumbnail generation, the current method of converting the entire pdf to a series of .png images is wasteful, and can bog down the consumer if dealing with a very long (page-wise) PDF.

This PR uses the convert bracket syntax to load only the first page, and get the thumbnail from that instead of doing all the pages.

@danielquinn

This comment has been minimized.

Copy link
Collaborator

commented Sep 9, 2018

This is a simple, and super-handy improvement! Thanks!

@danielquinn danielquinn merged commit 46cbd10 into the-paperless-project:master Sep 9, 2018

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details

@jat255 jat255 deleted the jat255:ENH_convert_only_one_page branch Sep 10, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.