Seidr ROC prints incorrect number of steps

bschiffthaler / seidr

Toolkit to create crowd networks

GNU General Public License v3.0

2 stars 1 forks source link

Closed bschiffthaler closed 4 years ago

bschiffthaler commented 4 years ago

The "logic" in seidr roc that decides whether to print a step or not is flawed, so running e.g.:

seidr roc -p 1000

results in more than 1000 output points. While this has no direct consequence for the ROC curve or the AUC/AUPR calculations, it needs to be fixed.

bschiffthaler commented 4 years ago

Using arma::linspace now to generate evenly split cutoff points. Should be fine now.