Biostrings icon indicating copy to clipboard operation
Biostrings copied to clipboard

consensusString and consensusMatrix error for legth-0 input

Open jan-glx opened this issue 11 months ago • 0 comments

Currently, consensusString fails for length-0 inputs:

Biostrings::consensusString(character(0))
#> Error in h(simpleError(msg, call)): error in evaluating the argument 'x' in selecting a method for function 'consensusString': 'x' has no element and 'width' is NULL

Created on 2023-07-29 by the reprex package (v2.0.1)

I believe it should return character(0) instead and at most warn.

Maybe, replacing error("'x' has no element and 'width' is NULL"); with return allocMatrix(INTSXP, 0, 0); is all that's needed?

jan-glx avatar Jul 29 '23 14:07 jan-glx