search

LandSciTech / ccviR

Implement NatureServe climate change vulnerability index in R
https://landscitech.github.io/ccviR/
Other
2 stars 1 forks source link

Look into bundling latex requirements with package so report generation is smoother #156

Open plestiodon opened 3 months ago

plestiodon commented 3 months ago

Report generation fails

A new version of TeX Live has been released. If you need to install or update any LaTeX packages, you have to upgrade TinyTeX with tinytex::reinstall_tinytex(repository = "illinois"). ! LaTeX Error: File `threeparttable.sty' not found.

! Emergency stop. <read *>

Warning: Error in : LaTeX failed to compile report.tex. See https://yihui.org/tinytex/r/#debugging for debugging tips. See report.log for more info. 3: runApp 2: print.shiny.appobj 1:

plestiodon commented 3 months ago

Upgrading TinyTex did fix the problem.

see24 commented 3 months ago

I am glad that you were able to fix it! I am not sure if there is anything we can do to avoid this issue. In the long term if the app is hosted on a server one of the benefits will be being able to control the environment and when things update.

plestiodon commented 3 months ago

Sarah,

Apologies if it wasn't good to post this on Github. It was an easy fix for me but for my students who lack familiarity with R this would likely not be something they could figure out on their own.

Pamela Rutherford Associate Professor Department of Biology Brandon University

COSEWIC – Amphibian and Reptile – Co-Chair

@.***

On Thu, Mar 21, 2024 at 8:01 AM Sarah Endicott @.***> wrote:

I am glad that you were able to fix it! I am not sure if there is anything we can do to avoid this issue. In the long term if the app is hosted on a server one of the benefits will be being able to control the environment and when things update.

— Reply to this email directly, view it on GitHub https://github.com/LandSciTech/ccviR/issues/156#issuecomment-2012236309, or unsubscribe https://github.com/notifications/unsubscribe-auth/BG23HMW5IZBZELELTHWAD2LYZLK4DAVCNFSM6AAAAABFAJEVOCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJSGIZTMMZQHE . You are receiving this because you authored the thread.Message ID: @.***>

see24 commented 3 months ago

No problem at all to log it here. It is good for any other users to see how they can solve this issue. I will keep this open because it would be good to look more into whether the LaTEX dependencies could be bundled with the R package in a way that would help to insulate it from this type of issue. This wouldn't work when installing from source from GitHub but could potentially work with something like runiverse or drat or similar where it is installed from a binary. Will look into this one of these days...