Seneca-ICTOER / Intro2C

Seneca College IPC144 Course Notes
https://ipc144.sdds.ca
Other
7 stars 52 forks source link

Generated PDF (GitHub Action): missing many images #168

Closed CameronGray1210 closed 2 years ago

CameronGray1210 commented 2 years ago

Generating the PDF on local machine will properly include all images, however, the automated PDF generation on GitHub fails to render many images.

humphd commented 2 years ago

Which one(s)? cc @TueNguyen2911

CameronGray1210 commented 2 years ago

Far too many to list (issue persists throughout all documents). It appears random and I don't see a pattern based on image type and syntax used in markdown - very perplexing. Here are a few to references to get things started. The generated PDF on GitHub misses the following images:

computer.md

information.md

Examples where images DO render correctly:

computers.md

compilers.md

The interesting thing is when the PDF process is generated locally (on my machine anyway) it works fine.

humphd commented 2 years ago

This sounds like https://github.com/kohheepeace/mr-pdf/issues/37

CameronGray1210 commented 2 years ago

Ahh.. yes that might very well be the cause. I was able to replicate the missing images even on a local build. When this happened, I then loaded all pages in the browser to build a cache, then generated the PDF and it worked. So there seems to be a link to how it uses the cached images (or the slow loading of them) to the rendering in the PDF.

humphd commented 2 years ago

The fix for this is to have the render wait on all images to load fully. I think it's likely to get fixed upstream.