preview mode not work

[hdeng88]

the html table will be shown in viewer mode not inline mode. works fine in knit html but not notebook.

[benjaminrich]

I don't really use notebook, is that RStudio specific, or a general problem?

[hdeng88]

I think it is Rstudio specific. but I have seen this happened to other packages but fixed afterwards quickly so I guess it is not a difficult fix.

Also could you provide a default option to output median [Q1,Q3]? I think this is more academically standard from a reviewer standpoint. If there are multiple variables needed to be presented in different format, how could I format them differently? say age for median[Q1,Q3] but weight for mean (SD).

[benjaminrich]

I will try to look into it, but I don't know when. If you can provide information on how notebook works in RStudio and how to feed it HTML that will be helpful.

As to your other question, you can. Here is an example:

library(table1)

dat <- data.frame(Age=runif(100, 40, 65), Weight=rnorm(100, 75, 10))

rndr <- function(x, name, ...) {
    what <- switch(name,
        Weight = "Mean (SD)",
        Age    = "Median [Q1, Q3]")

    parse.abbrev.render.code(c("", what))(x)
}

table1(~ Age + Weight, data=dat, render=rndr)

Edit: cleaned up the example slightly.

[benjaminrich]

This is working now in pull request #9. Do you want to test it?

[hdeng88]

sure. just checked. Somehow it did not work the way as we expected. Could u double check? Hao

…

On Wed, Jun 20, 2018 at 9:20 AM Benjamin Rich ***@***.***> wrote: This is working now in pull request #9 <#9>. Do you want to test it? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#7 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AOSPYyrPG3O-XguRp9OMNls4wH5Q0S3vks5t-kwugaJpZM4UezO9> .

[benjaminrich]

Are you using the right branch? The change has not been merged into the master branch yet, so you want to be installing with:

devtools::install_github("benjaminrich/table1#9")

[hdeng88]

oh sorry. I was being stupid to forget that. Hao

…

On Wed, Jun 20, 2018 at 11:04 AM Benjamin Rich ***@***.***> wrote: Are you using the right branch? The change has not been merged into the master branch yet, so you want to be installing with: devtools::install_github("benjaminrich/table1#9") — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#7 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AOSPY6mR8J1YGMa2-IIX7oVujfKrwbsvks5t-mR6gaJpZM4UezO9> .

[hdeng88]

Hi Benjamin, I had some trouble using the " devtools::install_github("benjaminrich/table1#9") " command. It says that:

devtools::install_github("benjaminrich/table1#9")

Downloading GitHub repo benjaminrich/table1@issue-#7 from URL https://api.github.com/repos/benjaminrich/table1/zipball/issue-#7 Installation failed: 404: Not Found (404) I got an error message after installing table1 from github. Error: package or namespace load failed for ‘table1’ in get(Info[i, 1], envir = env): lazy-load database '/Library/Frameworks/R.framework/Versions/3.4/Resources/library/Formula/R/Formula.rdb' is corrupt In addition: Warning message: In get(Info[i, 1], envir = env) : internal error -3 in R_decompress1

…

On Wed, Jun 20, 2018 at 12:00 PM, Hao Deng ***@***.***> wrote: oh sorry. I was being stupid to forget that. Hao On Wed, Jun 20, 2018 at 11:04 AM Benjamin Rich ***@***.***> wrote: > Are you using the right branch? The change has not been merged into the > master branch yet, so you want to be installing with: > > devtools::install_github("benjaminrich/table1#9") > > — > You are receiving this because you authored the thread. > Reply to this email directly, view it on GitHub > <#7 (comment)>, > or mute the thread > <https://github.com/notifications/unsubscribe-auth/AOSPY6mR8J1YGMa2-IIX7oVujfKrwbsvks5t-mR6gaJpZM4UezO9> > . >

[benjaminrich]

OK, my bad, it did not work as I expected. Try this instead:

devtools::install_github("benjaminrich/table1@issue-%237")

[hdeng88]

sorry still not working

…

On Thu, Jun 21, 2018 at 2:05 AM, Benjamin Rich ***@***.***> wrote: OK, my bad, it did not work as I expected. Try this instead: devtools::install_github("benjaminrich/table1", ref="issue-%237") — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#7 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AOSPYxCDxHOtE4lACdrMeSwyAukuZc08ks5t-zeWgaJpZM4UezO9> .

[benjaminrich]

Which one is still not working, the installation, or the preview in RStudio?

[hdeng88]

the preview. I have tried using result. = asis but it did Not work as well.

…

On Thu, Jun 21, 2018 at 9:47 AM Benjamin Rich ***@***.***> wrote: Which one is still not working, the installation, or the preview in RStudio? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#7 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AOSPY6069GHz2FKRDWCK7MaL7Ql14cFYks5t-6QBgaJpZM4UezO9> .

[benjaminrich]

It is working for me. Here's a screenshot:

[hdeng88]

I think it is related to ur rstudio or packages version? Any development version of packages you are using

…

On Thu, Jun 21, 2018 at 9:52 AM Benjamin Rich ***@***.***> wrote: It is working for me. Here's a screenshot: [image: screenshot_issue7] <https://user-images.githubusercontent.com/17572252/41723282-a74a1390-7538-11e8-9a9e-2c2a7f27ca89.png> — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#7 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AOSPYwFSgRH9J5p7DptOCHambm2z3ewMks5t-6UUgaJpZM4UezO9> .

[benjaminrich]

Here is the output of sessionInfo():

> sessionInfo()
R version 3.5.0 (2018-04-23)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows >= 8 x64 (build 9200)

Matrix products: default

locale:
[1] LC_COLLATE=English_United States.1252 
[2] LC_CTYPE=English_United States.1252   
[3] LC_MONETARY=English_United States.1252
[4] LC_NUMERIC=C                          
[5] LC_TIME=English_United States.1252    

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base     

other attached packages:
[1] table1_1.1    rmarkdown_1.9

loaded via a namespace (and not attached):
 [1] compiler_3.5.0  backports_1.1.2 magrittr_1.5    rprojroot_1.3-2
 [5] tools_3.5.0     htmltools_0.3.6 Rcpp_0.12.17    stringi_1.1.7  
 [9] knitr_1.20      Formula_1.2-3   stringr_1.3.1   digest_0.6.15  
[13] evaluate_0.10.1

Does that help?

[benjaminrich]

And my RStudio version is 1.1.453.

[hdeng88]

I will switch to Rstudio newest version to see if this is the key.

…

On Thu, Jun 21, 2018 at 10:31 AM Benjamin Rich ***@***.***> wrote: And my RStudio version is 1.1.453. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#7 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AOSPY4KaA1t-RNU8oYVVwmThHXdyjGjGks5t-64KgaJpZM4UezO9> .

[hdeng88]

Done. It is because of the version of rstudio

…

On Thu, Jun 21, 2018 at 10:32 AM Hao Deng ***@***.***> wrote: I will switch to Rstudio newest version to see if this is the key. On Thu, Jun 21, 2018 at 10:31 AM Benjamin Rich ***@***.***> wrote: > And my RStudio version is 1.1.453. > > — > You are receiving this because you authored the thread. > Reply to this email directly, view it on GitHub > <#7 (comment)>, > or mute the thread > <https://github.com/notifications/unsubscribe-auth/AOSPY4KaA1t-RNU8oYVVwmThHXdyjGjGks5t-64KgaJpZM4UezO9> > . >

[benjaminrich]

Excellent. Then I am closing the issue. Thanks.

[hdeng88]

sounds good! Great job!

…

On Thu, Jun 21, 2018 at 11:47 AM Benjamin Rich ***@***.***> wrote: Excellent. Then I am closing the issue. Thanks. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#7 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AOSPY7i_LF5iaOGlRgJWbu2fdRoOfoiQks5t-8AogaJpZM4UezO9> .