search

Bioconductor / RaggedExperiment

Matrix-like representations of mutation and CN data
https://bioconductor.org/packages/RaggedExperiment
4 stars 3 forks source link

qreduceSummarizedExperiment vs qreduceAssay argument names #19

Closed lwaldron closed 6 years ago

lwaldron commented 6 years ago

I was surprised that qreduceSummarizedExperiment has a different argument (simplify) than the corresponding argument of qreduceAssay (simplifyReduce). Shouldn't these functions have consistent argument names?

lwaldron commented 6 years ago

Same comment for the simplifyReduce / simplify arguments to disjoinAssay / disjoinSummarizedExperiment.

LiNk-NY commented 6 years ago

Thanks. I'll work on a fix for this.

LiNk-NY commented 6 years ago

Fixed in 96161709e569639b899b8d2de56d6e735006fe94