Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Warning for ties in stats #20

Merged
merged 3 commits into from
Feb 21, 2018
Merged

Warning for ties in stats #20

merged 3 commits into from
Feb 21, 2018

Commits on Feb 16, 2018

  1. Warning message for ties in the ranked stats 

    Print a message if there are ties in the metric and the percentage of it.    
This only for metric not equal to zero, as random order of genes equal to 0 it is no relevant.    

No idea which percentage of ties are aceptable, nor if it is relevant for the biological interpretation of the results. But the user should be aware of this when they are doing the analysis. As discussed in #18 and #19
    @ToledoEM
    ToledoEM authored Feb 16, 2018
    Configuration menu
    Copy the full SHA
    de796b3 View commit details
    Browse the repository at this point in the history

  2. use warning instead of message

    @ToledoEM
    ToledoEM authored Feb 16, 2018
    Configuration menu
    Copy the full SHA
    ba2b96d View commit details
    Browse the repository at this point in the history

Commits on Feb 21, 2018

  1. Add test for detection of ties 

    Testthat test for  "Ties detection in ranking works"
    @ToledoEM
    ToledoEM authored Feb 21, 2018
    Configuration menu
    Copy the full SHA
    1646d12 View commit details
    Browse the repository at this point in the history