search

lgessl / patroklos

An R package pipelining binary classification of survival
https://lgessl.github.io/patroklos/
Other
0 stars 0 forks source link

Speed up test suite #4

Closed lgessl closed 8 months ago

lgessl commented 8 months ago

Ways to do it:

  1. Use file IO function with strings via I() instead of actual reading from files.
  2. Use glmnet::glmnet() instead of glmnet::cv.glmnet() for fit(), prepare_and_fit() etc.
lgessl commented 8 months ago

Ad 2: Use suppressWarnings() instead since the warning has mathematical-theoretical, not a technical reason.

lgessl commented 8 months ago

Ad 2: glmnet::glmnet() is way slower than zeroSum::zeroSum() with zeroSum = FALSE, so use the latter one instead.

lgessl commented 8 months ago

Test suit has become substantially faster thanks to removing obsolete functions and their tests. On my platform, tests now take less than 10 sec.