jknowles / eeptools / 84

83%

master: 83%

Pull #39

travis-ci

Some awesome title

Why:

* In `max_mis`, there is an `if` predicate that checks if a class is of an invalid type. Because R objects can have multiple classes, this statement relies on the R semantics that the first value of the logical vector returned by the predicate is used. This is a feature R-core is looking to remove as it has considerable performance penalties. See https://developer.r-project.org/Blog/public/2018/10/12/conditions-of-length-greater-than-one/index.html for the discussion of testing for this behavior change and the following report showing `eeptools` failing a why: https://github.com/kalibera/rifcond/blob/master/outputs/eeptools.Rcheck/tests/test-all.Rout.fail

This change addresses the need by:

* This check is now wrapped in `any()`, which evaluates `TRUE` if any value in the logical vector is `TRUE`.

1 of 1 new or added line in 1 file covered. (100.0%)

374 of 474 relevant lines covered (78.9%)

22.03 hits per line