strobealign icon indicating copy to clipboard operation
strobealign copied to clipboard

Explicit error if too many sequences are used

Open luispedro opened this issue 1 year ago • 2 comments

Right now, strobealign only supports up to 2²⁴ sequences. If the user tries more, it would silently accept it, but later crash.

luispedro avatar Feb 09 '24 05:02 luispedro

Right now, strobealign only supports up to 2²⁴ sequences. If the user tries more, it would silently accept it, but later crash.

This was triggered when trying to map to the Greengenes database

https://ftp.microbio.me/greengenes_release/2022.10/ https://ftp.microbio.me/greengenes_release/2022.10/2022.10.seqs.fna.gz

Even a single read like the one below trigger a crash

@M05314:127:000000000-BWLLJ:1:1101:15267:1654 2:N:0:1
CCTGTTCGCTCCCCACGCTTTCGTCCCTCAGCGTCAATATTGTGCCAGAATGCTGCCTTCGCCATTGGTGTTCCTCCTGATATCTACGCATGTCACCGCTACACCAGGAATTCCACATTCCTCTCACATATTCTATTTTATCAGTTTTGAT
+
AAA1AF@1>AAAGG1A0EAFGGEHAAEGFCG1AAEE/F2FG2F2FF1CA0FBDED1BGFGFFE?AF1BFFCFHDGFFHB1FFGFGEEFE/?/BF2F@/EGEEB00/0//0BFG1>B1BGFEFHHGGFFD12BGH2FDFFFGG22GDD>@/F

luispedro avatar Feb 09 '24 05:02 luispedro

Thanks Luis! Approved on my end. For Marcel to merge.

ksahlin avatar Feb 11 '24 10:02 ksahlin

Thanks, good point!

marcelm avatar Feb 20 '24 08:02 marcelm