naoki-egami / DIDdesign

R package DIDdesign: Analyzing Difference-in-Differences Design
GNU General Public License v2.0
35 stars 7 forks source link

Fix dependency `future` #34

Closed szonszein closed 1 year ago

szonszein commented 1 year ago

Running: did_check(..., option = list(..., parallel = TRUE, ...))

Getting: Error: 'multiprocess' is not an exported object from 'namespace:future'

Installed future version 1.29.0. That fixes the error.

So, either fix package dependency to future version 1.29.0. or change function multiprocess to new version in 1.33.0.

naoki-egami commented 1 year ago

Thank you for the issue. We will fix this asap.

szonszein commented 1 year ago

Thank you! Probably related to parallel setup, another issue is with did_check_std when using more than three lags. See below my code and the Error message.

set.seed(1234) check <- did_check(

  • formula = y ~ treatment + post_treat,
  • data = data,
  • id_time = "time",
  • is_panel = FALSE,
  • option = list(n_boot = 2000, parallel = TRUE, id_cluster="id", lag = 1:6)
  • ) Error in est_boot_std[, i] : subscript out of bounds Calls: did_check ... did_check_std -> data.frame -> sd -> var -> is.data.frame Execution halted