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

Optimize apply_png_predictor #911

Closed
mitchelljkotler opened this issue Oct 13, 2023 · 0 comments · Fixed by #912
Closed

Optimize apply_png_predictor #911

mitchelljkotler opened this issue Oct 13, 2023 · 0 comments · Fixed by #912

Comments

@mitchelljkotler
Copy link
Contributor

I have a PDF which takes over 29 minutes to load on my computer, spending all of its time in apply_png_predictor. I noticed some easy ways to optimize that function, which resulted in a load time of less than 4 seconds, for a greater than 450x speed up. The PDF is probably unusual, but I think the optimized version should be helpful for any other similar PDFs. I will open a PR to close this issue with my optimized version of the method.

mitchelljkotler added a commit to MuckRock/pdfminer.six that referenced this issue Oct 13, 2023
@mitchelljkotler mitchelljkotler mentioned this issue Oct 13, 2023
5 tasks
mitchelljkotler added a commit to MuckRock/pdfminer.six that referenced this issue Oct 13, 2023
mitchelljkotler added a commit to MuckRock/pdfminer.six that referenced this issue Jan 22, 2024
github-merge-queue bot pushed a commit that referenced this issue Jun 24, 2024
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 a pull request may close this issue.

1 participant