- All functions must contain
@returns
- Most or all functions should contain
@examples
- All or most key functions should include unit tests for basic functionality.
-
DONE. Remove use of
:::
, replace with call to exported function, or internal function that performs equivalent task.- DONE:
grDevices:::.smoothScatterCalcDensity()
- is a wrapper forKernSmooth::bkde2D()
so can be replaced with internal Jam function. - DONE:
RColorBrewer:::namelist
- should be removed, RColorBrewer is not in Imports, but in Suggests. Use manually-encoded method to recognize valid RColorBrewer color names. (jamba-showcolors.R:52
) - DONE:
utils:::relist.default()
- remove reference in help docs, seejamba-string.r:2411
. - DONE:
base:::merge.data.frame()
- manually recognize argument names, seejamba.r:4632
.
- DONE:
-
Consider splitting large files into smaller counterparts, examples:
jamba.r
: 5,074 linesjamba-plots.r
: 3,341 linesjamba-string.r
: 2,629 linesjamba-colors.r
: 2,107 linesjamba-export.r
: 1,762 linesjamba-mixedSort.R
: 1,147 lines
-
Port
imageDefault()
to callimage.default()
- DONE: avoid
.External.graphics(graphics:::C_image, x, y, zi, col)
- DONE: avoid