Biostrings icon indicating copy to clipboard operation
Biostrings copied to clipboard

Fix: Removes length restriction on `writeXStringSet`, `writeQualityScaledXStringSet`

Open ahl27 opened this issue 1 year ago • 0 comments

writeXStringSet and writeQualityScaledXStringSet previously restricted inputs based on the width of sequences due to limitations in the internal I/O buffer. Now arbitrarily long sequences can be written, resulting in support for width > 200003 for XStringSet objects and also support for writing QualityScaledXStringSet objects with sequences longer than 200kbp.

More specific changes:

  • writeXStringSet now supports any integer value for width
  • writeXStringSet called with zero or negative values for width will now using unbounded width (i.e., each sequence written on a single line)
  • writeQualityScaledXStringSet now functions for sequences of arbitrary length, fixing Issue 177 (#117)
  • Unit test cases added to cover new functionality. Could consider adding more tests to cover other SolexaQuality and IlluminaQuality, as well as verifying that the output widths are the correct number of characters.

ahl27 avatar Dec 20 '24 22:12 ahl27