RPVote / eiCompare

Comparing ecological inference techniques
https://rpvote.github.io/voting-rights/
6 stars 7 forks source link

ei_iter() breaks when par_compute = TRUE #113

Closed aridf closed 4 years ago

aridf commented 4 years ago

Describe the bug Whenever ei_iter is run with it's parallel computing option enabled, it breaks with the error cannot find variable ei_sim.

To Reproduce Steps and code to reproduce the behavior:

  1. Run the ei vignette of the package through up to the ei_iter() step.
  2. Run the ei_iter() step, adding par_compute = TRUE as an additional argument

Expected behavior The function should run with iterations in parallel, without returning this error.

Environment information:

R version 4.0.2 (2020-06-22)
Platform: x86_64-apple-darwin19.5.0 (64-bit)
Running under: macOS Catalina 10.15.6

Matrix products: default
BLAS:   /System/Library/Frameworks/Accelerate.framework/Versions/A/Frameworks/vecLib.framework/Versions/A/libBLAS.dylib
LAPACK: /usr/local/Cellar/openblas/0.3.10_1/lib/libopenblasp-r0.3.10.dylib

locale:
[1] en_CA.UTF-8/en_CA.UTF-8/en_CA.UTF-8/C/en_CA.UTF-8/en_CA.UTF-8

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

other attached packages:
[1] eiCompare_2.4.9000 wru_0.1-9          ei_1.3-3           eiPack_0.2-1      

loaded via a namespace (and not attached):
  [1] fs_1.5.0           mcmcse_1.4-1       xopen_1.0.0        sf_0.9-5           usethis_1.6.1     
  [6] devtools_2.3.1     insight_0.9.0      doParallel_1.0.15  httr_1.4.2         rprojroot_1.3-2   
 [11] tools_4.0.2        backports_1.1.8    R6_2.4.1           tmvtnorm_1.4-10    KernSmooth_2.23-17
 [16] DBI_1.1.0          colorspace_1.4-1   withr_2.2.0        sp_1.4-2           tidyselect_1.1.0  
 [21] prettyunits_1.1.1  mnormt_2.0.1       processx_3.4.3     leaflet_2.0.3      compiler_4.0.2    
 [26] cli_2.0.2          expm_0.999-5       xml2_1.3.2         desc_1.2.0         sandwich_2.5-1    
 [31] bayestestR_0.7.2   scales_1.1.1       classInt_0.4-3     mvtnorm_1.1-1      callr_3.4.3       
 [36] stringr_1.4.0      digest_0.6.25      htmltools_0.5.0    pkgconfig_2.0.3    sessioninfo_1.1.1 
 [41] plotrix_3.7-8      htmlwidgets_1.5.1  rlang_0.4.7        rstudioapi_0.11    generics_0.0.2    
 [46] zoo_1.8-8          crosstalk_1.1.0.1  dplyr_1.0.2        magrittr_1.5       Matrix_1.2-18     
 [51] Rcpp_1.0.5         munsell_0.5.0      fansi_0.4.1        lifecycle_0.2.0    ucminf_1.1-4      
 [56] stringi_1.4.6      MASS_7.3-51.6      pkgbuild_1.1.0     grid_4.0.2         overlapping_1.6   
 [61] parallel_4.0.2     crayon_1.3.4       doSNOW_1.0.18      lattice_0.20-41    splines_4.0.2     
 [66] tmvnsim_1.0-2      knitr_1.29         ps_1.3.4           pillar_1.4.6       cubature_2.0.4.1  
 [71] codetools_0.2-16   stats4_4.0.2       pkgload_1.1.0      glue_1.4.1         msm_1.6.8         
 [76] data.table_1.13.0  remotes_2.2.0      vctrs_0.3.2        foreach_1.5.0      testthat_2.3.2    
 [81] rcmdcheck_1.3.3    tidyr_1.1.1        gtable_0.3.0       purrr_0.3.4        assertthat_0.2.1  
 [86] ggplot2_3.3.2      xfun_0.16          e1071_1.7-3        coda_0.19-3        roxygen2_7.1.1    
 [91] class_7.3-17       survival_3.1-12    tibble_3.0.3       snow_0.4-3         iterators_1.0.12  
 [96] gmm_1.6-5          tinytex_0.25       memoise_1.1.0      censusxy_1.0.0     ellipse_0.4.2     
[101] units_0.6-7        ellipsis_0.3.1    

Additional context Add any other context about the problem here.

aridf commented 4 years ago

Fixed by #114