1
94%
master: 94%

Ran 16 Apr 2026 12:23AM UTC

Files 55

Run time 4s

Badge

Embed ▾

Committed 16 Apr 2026 12:19AM UTC coverage: 92.923% (+0.003%) from 92.92%

Job # 24485183893.1

Build Type

push

github

Committed by

web-flow

Commit Message

Fix: collapse_substrings no longer re-appends absorbed short sequences (#139)

The inner loop over candidate superstrings reassigned `found_superstring` on
every iteration, so its final value only reflected whether the LAST candidate
contained the short sequence — not whether any of them did. When a shorter
VariantSequence was contained in an earlier (longer) result but not in a
later one, the short sequence's reads were both absorbed into the earlier
result AND the short sequence itself was appended to result_list, causing
the same reads to be counted twice in downstream assembly.

Add a `break` after the first superstring match so short sequences are
absorbed into exactly one superstring and not reappended.

Add two regression tests covering (a) the original scenario and (b) the
case where the short sequence has multiple valid superstrings.

Fixes #130

Coverage Stats

2022 of 2176 relevant lines covered (92.92%)

0.93 hits per line

openvax / isovar / 24485183893 / 1
94%
master: 94%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 24485183893.1

openvax / isovar / 24485183893 / 1 94% master: 94%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 24485183893.1

openvax / isovar / 24485183893 / 1
94%
master: 94%

README BADGES
x