Should we eventually refactor these to operate on ReadBases objects rather than strings?