Closed gerasy1987 closed 1 year ago
Will test integration of sampling and estimation functions again, and merge when successful. The re-write of the remaining sampling functions will be a new pull-request from this branch to main I guess.
@till-tietz . I am actually working on some edits for the sampling functions and will try to convert multi-/meta- study handlers to data table too for the meta-analyses we are planning. So closing this pull request for now.
@gerasy1987 sounds good...I'm about to wrap up the pps re-write using grouping and data.table. Will push this week.
Great, thanks @till-tietz . Also just a note as I am going through the sample_tls()
rewrite: If possible can we avoid using pipelines from magrittr
/dplyr
whatsoever. I have this idea that we can get rid of tidyverse completely eventually in the package and mixing data.table with pipelines is a bit confusing in my opinion.
Will test integration of sampling and estimation functions again, and merge when successful. The re-write of the remaining sampling functions will be a new pull-request from this branch to main I guess.