search

mixOmicsTeam / mixOmics

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

PR addressing Issue #111 #225

Closed Max-Bladen closed 1 year ago

Max-Bladen commented 2 years ago

As per #109, the user should be notified that the Y dataframe (in a DA context) contains NAs rather than just displaying an arbitrary error message

Max-Bladen commented 2 years ago

Ready for review @aljabadi.

I've gone through and seriously cleaned up each of the commits and the accompanying commit messages from the original PRs used to address this issue (#204, #223). I think this one should be fairly self-explanatory

Max-Bladen commented 1 year ago

Second set of commits on 13/12/2022 reflect initial set of commits, but replicated after the branch was rebased to master following PR #281.