search

relipmoc / skewer

MIT License
95 stars 17 forks source link

Changed tests for FASTQ output from qual.n to com.n #10

Closed wltrimbl closed 8 years ago

wltrimbl commented 9 years ago

This change prevents corrupt output (mixed fasta and fastq records) when encountering input fastq records with zero-length sequences.

com.n is a more reliable format indicator; it is definitiively zero with FASTA input and definitively nonzero with FASTQ input, regardless of the sequence length and quality legnth.

relipmoc commented 8 years ago

I've done that. Thank you!

wltrimbl commented 7 years ago

The problem, that zero-length fastq sequences in the output cause corrupt output persists.

wltrimbl commented 7 years ago

Sorry, I was mistaken, the current version does not have this bug; it is fixed.