-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for set.seed in parallel #15
Comments
Escalate to @lxw391 at the next meeting. @jamesban2015 reminded me that the supervised PCA would need the same fix (because of the randomly-generated parametric bootstrap samples). |
Could you try 10,000 permutations and see how much the results vary? hypothesis is that variation would decrease as number of permutations increase |
The results have nothing to do with the number of permutations, but rather the random number generation algorithm within R itself. This is not a trivial problem. It depends on the random number generator used. This article discusses parallelization with the Mersenne-Twister, Marsaglia Multicarry, and L’Ecuyer-CMRG random number generators: |
@lxw391 forwarded a Bioc-devel conversation to me:
I still see this as a non-critical enhancement for the upcoming version 1 release. |
another email threads about set.see in parallel setting |
See Section 6 of the
parallel
package vignette:https://stat.ethz.ch/R-manual/R-devel/library/parallel/doc/parallel.pdf
We need to add the ability to set seeds over multiple computing cores so that our AES-PCA and Supervised PCA function results are reproducible.
The text was updated successfully, but these errors were encountered: