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

Qiqqa PDF rendering error: PDF resize failed #9

Open
GerHobbelt opened this issue Jul 30, 2019 · 3 comments
Open

Qiqqa PDF rendering error: PDF resize failed #9

GerHobbelt opened this issue Jul 30, 2019 · 3 comments
Labels
🐛bug Something isn't working 🕵investigate Needs further analysis to find the root cause.
Milestone
v83

Comments

@GerHobbelt
Copy link
Collaborator

From the debug stats log (v80 github release on Windows 10/64):

20190730.192655 ERROR [39] There was an error while resizing a PDF page image

Utilities.GenericException: Error while rasterising page 1 at 187.5dpi of 'D:\Qiqqa\base\Guest\documents\2\26ABFE782ECFA6E4ACC9F93138A353B6F3126.pdf' --->
System.Runtime.InteropServices.SEHException: External component has thrown an exception.
at Utilities.PDF.Sorax.SoraxDLL.SPD_GetPageBitmap(IntPtr hDoc, IntPtr hDC, Int32 nPage, Int32 nRot, Single fDPI)
at Utilities.PDF.Sorax.SoraxPDFRendererDLLWrapper.GetPageByDPIAsImage_LOCK(HDOCWrapper hdoc, Int32 page, Single dpi)
--- End of inner exception stack trace ---
at Utilities.PDF.Sorax.SoraxPDFRendererDLLWrapper.GetPageByDPIAsImage_LOCK(HDOCWrapper hdoc, Int32 page, Single dpi)
at Utilities.PDF.Sorax.SoraxPDFRendererDLLWrapper.GetPageByHeightAsImage(String filename, String pdf_user_password, String pdf_owner_password, Int32 page, Double height)
at Utilities.PDF.Sorax.SoraxPDFRenderer.GetPageByHeightAsImage(Int32 page, Double height)
at Qiqqa.Documents.PDF.PDFControls.PDFRendererControlStats.ResizedPageImageItemThreadEntry(Object arg)
@GerHobbelt
Copy link
Collaborator Author

Adobe Acrobat DC reports an error on the PDF as well.

@GerHobbelt
Copy link
Collaborator Author

26ABFE782ECFA6E4ACC9F93138A353B6F3126.pdf

The offending PDF from the Qiqqa Guest/documents/2 directory.

@GerHobbelt
Copy link
Collaborator Author

GerHobbelt commented Aug 17, 2019 •
edited
Loading

Presentations in PDF format don't scale correctly in Qiqqa views and sniffer

Other example PDF files which do not size correctly in the GoogleSniffer window: (these scale correctly if you jiggle the splitter-bar between bibtex and PDF viewer panes left&right a little)

7B599A0AAFA19F0AE415129A185AB13FA42A3.pdf

C7A730925F25BA3DA70B0F7A21CD2B08EEE27.pdf

F2F3FB531C7079234E3647E962C32ECA12675F.pdf

46AA33B5235AB3E41D77B4B0BAF3CA74890901C.pdf

@GerHobbelt GerHobbelt added 🐛bug Something isn't working 🤔question Further information is requested or this is a support question 🕵investigate Needs further analysis to find the root cause. and removed 🤔question Further information is requested or this is a support question labels Oct 4, 2019
@GerHobbelt GerHobbelt added this to the v83 milestone Oct 9, 2019
GerHobbelt added a commit to GerHobbelt/qiqqa-open-source that referenced this issue Jan 18, 2021
@GerHobbelt
- preparing the PDFRenderer to use latest MuPDF mudraw tool. (submodu…
306dc14 
…le updated: -q quiet mode patches, ...)

  Going to use binary PNG transmission over stdout as working with a temporary file would quickly wear out any SSD serving as temp disk: one image rendered for every page, thumbnail of a page, etc.etc.: that's a lot of images rendered and saved to temp disk space if we used that as intermediate storage. :-(

- adding width next to height as a render criterium: this should ultimately fix a long-standing problem I have with Qiqqa rendering 'PowerPoint presentation PDFs' in a crazy way: those pages SHOULD be fitted to the screen as well as all other pages.

That's about jimmejardine#7, jimmejardine#9, jimmejardine#209, jimmejardine#280 and a part of the work towards jimmejardine#289
GerHobbelt added a commit to GerHobbelt/qiqqa-open-source that referenced this issue Jan 19, 2021
@GerHobbelt
migrate the height/width logic around the PDF page renderer from doub…
5459dc7 
…le to int. This should help improve cache performance a bit as now == equality comparisons should actually deliver instead of possibly maybe matching.

Also fixed a width edit I missed (the old cache and calc stuff only reckoned with height :-( -- jimmejardine#9 )

Added cache retrieval code which picks a higher rez image when already present: this should ultimate reduce the pressure on the PDF renderer, of course depending on the order of incoming requests (thumbnails in side panel vs full rez main page views)
This was referenced Feb 6, 2021
Open
Closed
@GerHobbelt GerHobbelt mentioned this issue Feb 27, 2021
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🐛bug Something isn't working 🕵investigate Needs further analysis to find the root cause.
Projects
None yet
Milestone
v83
Development

No branches or pull requests
1 participant
@GerHobbelt