You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I am trying to add support for the parabar backend, but I need to check with you first where parallelization is currently needed in bootnet. Can I safely assume that, at commit 9df9eab, the following place are the only places where a parallel backend is required?
The parSim function has actually been ported to the parSim package, which I use a lot for simulation studies. I should also make that imported in bootnet instead of having this old code in bootnet. Other than that, indeed only the bootnet function I think.
The PR with support for parabar backends need to in the context of the parSim package.
The bootnet::bootnet function needs to be updated to use parSim::parSim.
I'll work on (1) and also give (2) a try. Since there are no tests for bootnet, it would be handy if you could take a closer look at the changes for (2). After these are done, I think we can also finally close #82.
Hi Sacha,
I am trying to add support for the
parabar
backend, but I need to check with you first where parallelization is currently needed inbootnet
. Can I safely assume that, at commit 9df9eab, the following place are the only places where a parallel backend is required?In
R/bootnet.R
:bootnet/R/bootnet.R
Lines 612 to 613 in 9df9eab
bootnet/R/bootnet.R
Lines 788 to 790 in 9df9eab
In
R/parSim.R
:bootnet/R/parSim.R
Lines 64 to 65 in 9df9eab
bootnet/R/parSim.R
Lines 95 to 96 in 9df9eab
Best,
Mihai
The text was updated successfully, but these errors were encountered: