mixOmicsTeam / mixOmics

Development repository for the Bioconductor package 'mixOmics '
http://mixomics.org/
157 stars 52 forks source link

PR addressing Issue #275 #276

Closed Max-Bladen closed 1 year ago

Max-Bladen commented 1 year ago

As written in the associated Issue, presence of NAs in X parameter of tune.spca() was resulting in ambiguous error.

This PR includes a slight refactoring of some of the source code, as well as a check at the start of the function. If there are NAs, they are replaced with 0 and a warning is raised informing users of this imputation. Additionally, added new test file for tune.spca() to ensure coverage is maintained