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

Replace options() with withr::with_options() throughout #51

Open
jbullock35 opened this issue Dec 27, 2020 · 0 comments
Open

Replace options() with withr::with_options() throughout #51

jbullock35 opened this issue Dec 27, 2020 · 0 comments

Comments

@jbullock35
Copy link
Owner

jbullock35 commented Dec 27, 2020

https://stackoverflow.com/questions/65466324/why-are-sapply-and-options-undesirable

See zzz.R in my package.

Perhaps also replace this code from latexTablePDF() with an option:

  # CHECK FOR PDFLATEX
  # This is a time-consuming operation. So if we find pdflatex the first time,  
  # we create the ".pdflatex_found" variable in the user's environment. In 
  # future runs of latexTablePDF(), the check aborts if ".pdflatex_found" 
  # exists.  [2020 01 01]
  if (writePDF && !exists(".pdflatex_found")) {
    if (!nzchar(Sys.which("pdflatex")))  {
      stop("pdflatex doesn't seem to be on your path. A PDF file can't be created.")
    }
    else {
      assign(".pdflatex_found", TRUE, envir = parent.frame())
    }
  }
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

No branches or pull requests

1 participant