Skip to content

Latest commit

 

History

History
74 lines (49 loc) · 2.58 KB

cran-comments.md

File metadata and controls

74 lines (49 loc) · 2.58 KB

shapr 1.0.2 (Patch release)

  • (Resubmission of exactly the same package as submitted Jan 22nd. During check there seemed to be installation issues with the dials and rsample package (unrelated to my package). Unfortunately my emails commenting on this seems to have not reached the CRAN team, which unfortunately removed shapr 1.0.1. on CRAN. See below for original patch release comments.)

  • Fix test error in prepare_data_gaussian_cpp seen on clang-UBSAN gcc-UBSAN compliers in CRAN checks. Turned out to be an actual bug for an edge case.

  • Other minor fixes to GH repo, pkgdown site, python wrapper ++ unrelated to the actual shapr CRAN package. See NEWS.md for details.

Test environments

The majority of our tests rely uses snapshots to check results reproducibility. These tests are run locally and remotely with GHA (see below) without errors. The win-builder and R-hub tests are run without snapshots tests (to replicate CRAN testing)

  • GHA (ubuntu-latest), R-version: devel, release, oldrel-1, oldrel-2
  • GHA (windows-latest), R-version: release
  • GHA (macOS-latest), R-version: release
  • devtools-win-builder, R-versions: devel, release, oldrelease
  • devools-mac-builder, R-versions: release
  • R-hub (ubuntu-latest): R-version: devel
  • R-hub (windows-latest): R-version: devel
  • R-hub (clang-asan): R-version: devel

R CMD check results

There were no ERRORs, WARNINGs

There were in total 2 NOTES

NOTE (multiple platforms):

  • checking installed package size ... NOTE installed size is 7.1Mb sub-directories of 1Mb or more: doc 4.2Mb libs 1.3Mb

The package is growing in size, uses more complied code, and the documentation is comprehensive.

NOTE (win-builder):

  • checking CRAN incoming feasibility ... [20s] NOTE Maintainer: 'Martin Jullum Martin.Jullum@nr.no'

Days since last update: 6

Correct. I was required to fix the issue by the CRAN team.

Downstream dependencies

There is 2 downstream dependency (PPtreeregViz, SEMdeep) of shapr. In addition, the package shapviz provides additional plotting functionality without stating an explicit dependency (I believe shapr actually should have be listed under Suggests?)

The 2 packages fails on R CMD checks, but I have submitted PRs to fix them.

PPtreeregViz

SEMdeep

shapviz

*Passes the checks