This pull request exposes an option to allow for ignoring sequence record description mismatches. It came about from applying q2-demux to all of the sequencing runs from the AGP, and observing that at least one sequencing run had mismatches in the description fields but the IDs otherwise matched.
One note, the seqs iterator is implicitly constructed so I could not see a way to easily pass in a parameter. Instead, the flag on the instance is toggled within the API methods.
Separately, the BarcodePairedSequenceFastqIterator object did not have tests for description mismatch so those were added.
This pull request exposes an option to allow for ignoring sequence record description mismatches. It came about from applying q2-demux to all of the sequencing runs from the AGP, and observing that at least one sequencing run had mismatches in the description fields but the IDs otherwise matched.
One note, the
seqsiterator is implicitly constructed so I could not see a way to easily pass in a parameter. Instead, the flag on the instance is toggled within the API methods.Separately, the
BarcodePairedSequenceFastqIteratorobject did not have tests for description mismatch so those were added.