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.

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

Add kableExtra package, please #74

Open
ciaranmci opened this issue Mar 10, 2022 · 2 comments · May be fixed by #167
Open

Add kableExtra package, please #74

ciaranmci opened this issue Mar 10, 2022 · 2 comments · May be fixed by #167

Comments

@ciaranmci
Copy link

I want to produce a publicaiton-ready table using the kableExtra package in R. I previously requested installation of the magick package because the developers of kable libraries mentioned it as a dependency...but of course I never requested installation of kableExtra.

I haven't needed the other dependency (webshot package) to produce the publication-ready table when running a local instance of R (not in the OpenSAFELY docker). Would it be worth installing that too, or is its webpage functionality a No-No for containers?

@ciaranmci
Copy link
Author

I've noticed that the kableExtra and webshot packages have been installed. Thank you.

My local OpenSAFELY docker container still fails to save a kable table object using the kableExtra::save_kable().
The guidance on feature dependency notes that the machine might need to run webshot::install_phantomjs():
image

Trying to call webshot::install_phantomjs() obviously throws an error in my local OpenSAFELY docker container (I assume because a call is being made to download something from outside).

Am I correct to assume that my lovely kable tables are probably not possible if kableExtra needs PhantomJS installed in the environment?

Log below:
image

@bloodearnest
Copy link
Member

Yes, that sounds like the case. We may be able to install it into the docker image, depending on how big it is. I'll look into it.

@remlapmot remlapmot linked a pull request Dec 4, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants