PeteHaitch / DelayedMatrixStats

A port of the matrixStats API to work with DelayedMatrix objects from the DelayedArray package
Other
15 stars 7 forks source link

Devel matrix stats 1.0.0 resync #96

Closed PeteHaitch closed 1 year ago

PeteHaitch commented 1 year ago

Passes R CMD check, albeit with warnings and notes to tidy up, on my macOS (M2) laptop on R 4.3.1 with forthcoming MatrixGenerics v1.13.1 (https://github.com/Bioconductor/MatrixGenerics/pull/33) and matrixStats v1.0.0.

── R CMD check results ──────────────────────────────────── DelayedMatrixStats 1.23.1 ────
Duration: 3m 4.3s

❯ checking for missing documentation entries ... WARNING
  Undocumented S4 methods:
    generic 'colAnyMissings' and siglist 'matrix'
    generic 'rowAnyMissings' and siglist 'matrix'
  All user-level objects in a package (including S4 classes and methods)
  should have documentation entries.
  See chapter ‘Writing R documentation files’ in the ‘Writing R
  Extensions’ manual.

❯ checking dependencies in R code ... NOTE
  Missing object imported by a ':::' call: ‘DelayedArray:::.reduce_array_dimensions’
  Unexported objects imported by ':::' calls:
    ‘DelayedArray:::.get_ans_type’ ‘DelayedArray:::RleArraySeed’
    ‘DelayedArray:::get_Nindex_lengths’ ‘DelayedArray:::set_dim’
    ‘DelayedArray:::set_dimnames’ ‘DelayedArray:::subset_by_Nindex’
    ‘DelayedArray:::to_linear_index’
    See the note in ?`:::` about the use of this operator.

0 errors ✔ | 1 warning ✖ | 1 note ✖
Error: R CMD check found WARNINGs
Execution halted
PeteHaitch commented 1 year ago

Merged and pushed to Bioconductor's git server.