katehoffshutta
katehoffshutta
Sorry to miss this mention earlier! Thanks for merging. This is ready for devel; needs work before master. In particular, it outputs a very large file with all the networks...
Thanks @marouenbg @blawney! @marouenbg no working example needed, I asked @blawney to raise this so we can correct the ambiguity of the exception. As Marouen mentioned, the problem and fix(workaround)...
Currently I am not planning to implement the default as fall back to `estimate_p_values_mc` at this point because it is somewhat time-consuming. We could provide the user an estimated run...
Note that this can also occur in LIONESS-DRAGON. I will implement similar fixes here.
Thanks, Viola - this is accurate. We will try to get this in for the next release.
@taraeicher Please keep this issue open, it has not been resolved. You can assign it to me, I will work on it! Thanks for bringing this up again.
@taraeicher I don't think we ever had fgsea imported, actually. I added it here and removed the previous `library(fgsea)` code, replacing it with a roxygen import statement. Look okay?