• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

jknowles / eeptools / 84 / 2
83%
master: 83%

Build:
Build:
LAST BUILD BRANCH: fix-logical-check-warning
DEFAULT BRANCH: master
Ran 14 Oct 2018 12:34AM UTC
Files 21
Run time 1s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

14 Oct 2018 12:20AM UTC coverage: 78.903%. Remained the same
NOT_CRAN="false"

Pull #39

travis-ci

web-flow
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`.
Pull Request #39: Removes vector response from if predicate to help R-core move this to an error condition

374 of 474 relevant lines covered (78.9%)

7.34 hits per line

Source Files on job 84.2 (NOT_CRAN="false")
  • Tree
  • List 0
  • Changed 1
  • Source Changed 1
  • Coverage Changed 0
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Build 82
  • Travis Job 84.2
  • 616e1679 on github
  • Prev Job for NOT_CRAN="false" on fix-logical-check-warning (#83.2)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc