(s)PCA Build using `RSpectra::svds()`

mixOmicsTeam / mixOmics

Development repository for the Bioconductor package 'mixOmics '

http://mixomics.org/

153 stars 51 forks source link

(s)PCA Build using `RSpectra::svds()` #251

Closed Max-Bladen closed 1 year ago

Max-Bladen commented 1 year ago

Imported the RSpectra::svds() function to both pca() and spca().

For both, the output values are consistent.
In the case of the former, run time is significantly improved.
In the case of the latter, runtime is only marginally improved. This is due to the spca() function being highly inefficient generally. The SVD calculations are only one aspect of this inefficiency.

Max-Bladen commented 1 year ago

Required rebase to master branch after resolution of Issue #252 (via PR #253)

Max-Bladen commented 1 year ago

Really weird issue arising preventing the checks from passing. Honestly, not worth the effort in trying to resolve as this is a PR generated for a single user from the MIG lab.