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

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

Open
gacolitti opened this issue Jul 19, 2022 · 1 comment

Comments

@gacolitti
Copy link

gacolitti commented Jul 19, 2022

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

@nacnudus
Copy link
Owner

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

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

2 participants