While reviewing the package, three dependency-declaration issues surfaced:
-
dplyr is in Suggests but used unconditionally in package code. dplyr:: is called in 13 files in R/ (e.g. R/hc.R, R/hcp-ma.R, R/gof.R, R/tidy.R, R/glance.R, R/plot-cdf.R, ...). A package whose exported functions call dplyr:: must declare it in Imports; otherwise R CMD check flags it and the functions fail at runtime when dplyr is not installed. → move dplyr to Imports.
-
abind is in Imports but never used. The only reference is #' @importFrom abind abind in R/ssdtools-package.R; abind() is never called. → remove abind from Imports and drop the @importFrom.
-
reshape2 is in Suggests but never used anywhere in R/, tests/, or vignettes/. reshape2 is also a superseded/discouraged package. → remove from Suggests.
Note: plyr is retained — it is genuinely used (plyr::summarise() in R/ggproto.R). Replacing it has different semantics and is out of scope here.
These are low-risk declaration changes with no change to package behaviour. A PR follows.
While reviewing the package, three dependency-declaration issues surfaced:
dplyris inSuggestsbut used unconditionally in package code.dplyr::is called in 13 files inR/(e.g.R/hc.R,R/hcp-ma.R,R/gof.R,R/tidy.R,R/glance.R,R/plot-cdf.R, ...). A package whose exported functions calldplyr::must declare it inImports; otherwiseR CMD checkflags it and the functions fail at runtime whendplyris not installed. → movedplyrtoImports.abindis inImportsbut never used. The only reference is#' @importFrom abind abindinR/ssdtools-package.R;abind()is never called. → removeabindfromImportsand drop the@importFrom.reshape2is inSuggestsbut never used anywhere inR/,tests/, orvignettes/.reshape2is also a superseded/discouraged package. → remove fromSuggests.Note:
plyris retained — it is genuinely used (plyr::summarise()inR/ggproto.R). Replacing it has different semantics and is out of scope here.These are low-risk declaration changes with no change to package behaviour. A PR follows.