-
Notifications
You must be signed in to change notification settings - Fork 10.1k
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
Update jpx decoder #18257
Update jpx decoder #18257
Conversation
The decoder has been updated to reduce the number of copies after having decoded an image: mozilla/pdf.js.openjpeg@b4ba3b3
/botio test |
From: Bot.io (Windows)ReceivedCommand cmd_test from @calixteman received. Current queue size: 0 Live output at: http://54.193.163.58:8877/bb1165482121464/output.txt |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @calixteman received. Current queue size: 0 Live output at: http://54.241.84.105:8877/0c11bfa5c26d057/output.txt |
From: Bot.io (Linux m4)FailedFull output at http://54.241.84.105:8877/0c11bfa5c26d057/output.txt Total script time: 28.59 mins
Image differences available at: http://54.241.84.105:8877/0c11bfa5c26d057/reftest-analyzer.html#web=eq.log |
The reference test image difference seem to be limited to the known intermittents, but one error is reported (see below) and that one seems new given that I haven't seen it in earlier runs. Could that be a regression of this patch, or is it a new intermittent?
|
I don't manage to have an issue while I'm loading this pdf. |
From: Bot.io (Windows)FailedFull output at http://54.193.163.58:8877/bb1165482121464/output.txt Total script time: 44.30 mins
Image differences available at: http://54.193.163.58:8877/bb1165482121464/reftest-analyzer.html#web=eq.log |
Thanks for checking! Let's try again in that case and see if it disappears, but given that the Windows bot also didn't fail on this it does look like an intermittent: /botio-linux browsertest |
From: Bot.io (Linux m4)ReceivedCommand cmd_browsertest from @timvandermeij received. Current queue size: 0 Live output at: http://54.241.84.105:8877/2f41ad5046e3b05/output.txt |
We saw that one fail in PR #18167, where I was a bit worried about the changes there somehow causing it. (Since the image in question should be using a /FlateDecode filter.)
Well, if this is intermittently failing it might be quite difficult to reproduce (reliably) locally unfortunately... |
From: Bot.io (Linux m4)FailedFull output at http://54.241.84.105:8877/2f41ad5046e3b05/output.txt Total script time: 20.07 mins
Image differences available at: http://54.241.84.105:8877/2f41ad5046e3b05/reftest-analyzer.html#web=eq.log |
That's good to know. Unfortunately then it's apparently so intermittent that we haven't seen it in two weeks time, and I'd also not really have an idea how to reproduce it locally other than perhaps trying to repeatedly open/close the file until it happens. However, the run above indeed confirmed that it's not caused by this particular PR, so I'll make a follow-up issue for it. |
Thanks! |
The decoder has been updated to reduce the number of copies after having decoded an image:
mozilla/pdf.js.openjpeg@b4ba3b3