IRanges icon indicating copy to clipboard operation
IRanges copied to clipboard

Support 'compress=NA' in AtomicList constructors

Open hpages opened this issue 2 years ago • 0 comments

It would do compress=TRUE if the input is not too big, otherwise it would fallback to compress=FALSE. For example:

RleList(Rle(8, 2e9), Rle(5, 200), compress=NA)

would do RleList(Rle(8, 2e9), Rle(5, 200), compress=TRUE) i.e. return a CompressedRleList object.

But:

RleList(Rle(8, 2e9), Rle(5, 2e9), compress=NA)

would do RleList(Rle(8, 2e9), Rle(5, 2e9), compress=FALSE) i.e. return a SimpleRleList object.

Then compress=NA should probably be made the new default (right now it's compress=TRUE, which is problematic because it fails when the input is too big).

See https://github.com/Bioconductor/VariantAnnotation/issues/71#issuecomment-1501066775 for some background.

hpages avatar Apr 19 '23 19:04 hpages