tidyxl icon indicating copy to clipboard operation
tidyxl copied to clipboard

Hidden columns sometimes cause R session to crash with calling tidyxl::xlsx_cells()

Open gacolitti opened this issue 1 year ago • 1 comments

Using test.xlsx I can reproduce an R session crash every time:

tidyxl::xlsx_cells(path = "test.xlsx")

test.xlsx

CleanShot 2022-07-19 at 10 07 40@2x

If I unhide column "B", the R session does not crash.

Using test2.xlsx, the hidden column does not produce a crash:

tidyxl::xlsx_cells(path = "test2.xlsx")

test2.xlsx

gacolitti avatar Jul 19 '22 17:07 gacolitti

Thanks for reporting this. Apologies for this slow reply. I could reproduce it a few days ago, but it now works with the following sessionInfo(). If you reply with your session info, I'll see if I can reproduce it again and fix it.

R version 4.2.1 (2022-06-23)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Arch Linux

Matrix products: default
BLAS/LAPACK: /usr/lib/libopenblas_haswellp-r0.3.20.so

locale:
 [1] LC_CTYPE=en_GB.UTF-8       LC_NUMERIC=C              
 [3] LC_TIME=en_GB.UTF-8        LC_COLLATE=en_GB.UTF-8    
 [5] LC_MONETARY=en_GB.UTF-8    LC_MESSAGES=en_GB.UTF-8   
 [7] LC_PAPER=en_GB.UTF-8       LC_NAME=C                 
 [9] LC_ADDRESS=C               LC_TELEPHONE=C            
[11] LC_MEASUREMENT=en_GB.UTF-8 LC_IDENTIFICATION=C       

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

other attached packages:
[1] nvimcom_0.9-131 devtools_2.4.3  usethis_2.1.5  

loaded via a namespace (and not attached):
 [1] Rcpp_1.0.9        magrittr_2.0.3    pkgload_1.2.4    
 [4] R6_2.5.1          rlang_1.0.2       fastmap_1.1.0    
 [7] tools_4.2.1       pkgbuild_1.3.1    sessioninfo_1.2.2
[10] cli_3.3.0         withr_2.5.0       ellipsis_0.3.2   
[13] remotes_2.4.2     tidyxl_1.0.7.9000 rprojroot_2.0.3  
[16] lifecycle_1.0.1   crayon_1.5.1      brio_1.1.3       
[19] processx_3.5.3    purrr_0.3.4       callr_3.7.0      
[22] fs_1.5.2          ps_1.7.0          testthat_3.1.4   
[25] memoise_2.0.1     glue_1.6.2        cachem_1.0.6     
[28] compiler_4.2.1    desc_1.4.1        prettyunits_1.1.1

nacnudus avatar Jul 26 '22 20:07 nacnudus