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.

Sign up for GitHub

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

Jump to bottom

Slow performance #2504

Closed
okbtsnr opened this issue Dec 24, 2012 · 1 comment · Fixed by #12001
Closed

Slow performance #2504

okbtsnr opened this issue Dec 24, 2012 · 1 comment · Fixed by #12001
Labels
core performance

Comments

@okbtsnr
Copy link

okbtsnr commented Dec 24, 2012

http://dl-ctlg.panasonic.com/jp/manual/sd/sd_rbm1000_0.pdf
@timvandermeij timvandermeij changed the title Garble and Slow Slow performance Mar 18, 2014
@timvandermeij
Copy link
Contributor

[Edited this issue to reflect the current state]

The server serving the PDF is very slow, but once you have saved it and load it manually, you'll see that it's not too bad, but still each page takes approx. 2 seconds to render, which could be better.

nnethercote added a commit to nnethercote/pdf.js that referenced this issue Aug 15, 2014
@nnethercote
Right-size the array in getSampleArray().
bcc31b6 
This allows the JS engine to do a better job of allocating the right
number of elements for the array, avoiding some resizings. For the PDF
in mozilla#2504, this avoids 100s of MiBs of allocations in Firefox.
@nnethercote nnethercote mentioned this issue Aug 15, 2014
Merged
nnethercote added a commit to nnethercote/pdf.js that referenced this issue Aug 15, 2014
@nnethercote
Ignore setGState no-ops.
9674abc 
For the document in mozilla#2504, 11% of the ops are `setGState` with a
`gStateObj` that is an empty array, which is a no-op. This is possible
because we ignore various setGState keys (OP, OPM, BG, etc).

This change prevents these ops from being inserted into the operator
list.
@nnethercote nnethercote mentioned this issue Aug 15, 2014
Merged
@Snuffleupagus Snuffleupagus mentioned this issue Aug 20, 2019
Merged
@Snuffleupagus Snuffleupagus mentioned this issue Jun 14, 2020
Merged
This was referenced Feb 2, 2023
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
core performance
Projects
None yet
Milestone
No milestone
2 participants
@timvandermeij @okbtsnr