spgarbet / tangram

Table Grammar package for R
67 stars 3 forks source link

summary_table should be tangram #24

Closed emilelatour closed 7 years ago

emilelatour commented 7 years ago

Hello,

I just want to point out that in your vignettes and examples, the function call summary_table() should be tangram()

At least that's what I had to do for it to work

> sessionInfo()
R version 3.4.0 (2017-04-21)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows 7 x64 (build 7601) Service Pack 1

Matrix products: default

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

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

other attached packages:
 [1] tangram_0.2.6        magrittr_1.5         digest_0.6.12        base64enc_0.1-3      stringr_1.2.0       
 [6] R6_2.2.1             ggthemes_3.4.0       wesanderson_0.3.2    Hmisc_4.0-3          Formula_1.2-1       
[11] survival_2.41-3      hrbrthemes_0.3.2     bindrcpp_0.1         rstan_2.15.1         StanHeaders_2.15.0-1
[16] forcats_0.2.0        pscl_1.4.9           lattice_0.20-35      MASS_7.3-47          scales_0.4.1.9000   
[21] dplyr_0.7.0          purrr_0.2.2.2        readr_1.1.1          tidyr_0.6.3          tibble_1.3.3        
[26] ggplot2_2.2.1        tidyverse_1.1.1     

loaded via a namespace (and not attached):
 [1] httr_1.2.1            jsonlite_1.5          splines_3.4.0         modelr_0.1.0          assertthat_0.2.0     
 [6] stats4_3.4.0          latticeExtra_0.6-28   pander_0.6.0          cellranger_1.1.0      yaml_2.1.14          
[11] Rttf2pt1_1.3.4        backports_1.1.0       glue_1.1.0            extrafontdb_1.0       checkmate_1.8.2      
[16] RColorBrewer_1.1-2    rvest_0.3.2           colorspace_1.3-2      htmltools_0.3.6       Matrix_1.2-10        
[21] plyr_1.8.4            psych_1.7.5           broom_0.4.2           haven_1.0.0           htmlTable_1.9        
[26] janitor_0.3.0         nnet_7.3-12           lazyeval_0.2.0        mnormt_1.5-5          readxl_1.0.0         
[31] evaluate_0.10         nlme_3.1-131          xml2_1.1.1            foreign_0.8-68        data.table_1.10.4    
[36] tools_3.4.0           hms_0.3               munsell_0.4.3         cluster_2.0.6         compiler_3.4.0       
[41] rlang_0.1.1           grid_3.4.0            rstudioapi_0.6.0.9000 htmlwidgets_0.8       labeling_0.3         
[46] rmarkdown_1.6         gtable_0.2.0          codetools_0.2-15      inline_0.3.14         reshape2_1.4.2       
[51] gridExtra_2.2.1       lubridate_1.6.0       knitr_1.16            extrafont_0.17        bindr_0.1            
[56] rprojroot_1.2         stringi_1.1.5         parallel_3.4.0        Rcpp_0.12.11          rpart_4.1-11         
[61] acepack_1.4.1        
spgarbet commented 7 years ago

That is correct for the current master branch, it's not an official release. I'm in the middle of a big refactor. Trying to finish up before UseR! 2017. I'll leave this open till all of this breakage is fixed.

spgarbet commented 7 years ago

Thank you for the poke. Sometimes that's all it takes. This is now fixed.

emilelatour commented 7 years ago

Awesome!! Thanks Shawn! Looking forward to trying out you package more!!

From: Shawn Garbett [mailto:notifications@github.com] Sent: Tuesday, June 27, 2017 7:42 AM To: spgarbet/tangram tangram@noreply.github.com Cc: Emile Latour latour@ohsu.edu; Author author@noreply.github.com Subject: Re: [spgarbet/tangram] summary_table should be tangram (#24)

Thank you for the poke. Sometimes that's all it takes. This is now fixed.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/spgarbet/tangram/issues/24#issuecomment-311379984, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AYOC9C5gMsaRWRqXOZ2K4XQZZYnPGxmHks5sIRSrgaJpZM4OGBnR.

spgarbet commented 7 years ago

I'm pushing towards the 0.3 release. Moving LaTeX and RTF rendering back to 0.3.1. It's important to get this refactor stable. CRAN check is now on documentation issues.