Skip to content

Fix push up push_up_unary_recombinant_mutations issue - #631

Merged
jeromekelleher merged 2 commits into
mainfrom
fix-push-up-problem
Oct 9, 2026
Merged

jeromekelleher merged 2 commits into
mainfrom
fix-push-up-problem

Conversation

@jeromekelleher

Copy link
Copy Markdown
Member

No description provided.

Count all child edges of a recombinant node, not just full-span ones,
when deciding whether it is unary. Previously a recombinant with one
full-span child and other partial-span children was treated as unary,
so mutations pushed up onto it were inherited by the other children,
giving a bad mutation parent error.
This was added as a workaround for the push_up_unary_recombinant_mutations
bug, which is now fixed.
@codecov-commenter

codecov-commenter commented Oct 9, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 88.68%. Comparing base (2e3b022) to head (bf5395e).
⚠️ Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
sc2ts/inference.py 85.71% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #631   +/-   ##
=======================================
  Coverage   88.68%   88.68%           
=======================================
  Files          12       12           
  Lines        4320     4322    +2     
  Branches      620      621    +1     
=======================================
+ Hits         3831     3833    +2     
  Misses        361      361           
  Partials      128      128           
Flag Coverage Δ
python-tests 88.68% <90.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jeromekelleher
jeromekelleher merged commit f47f13a into main Oct 9, 2026
9 checks passed
@jeromekelleher
jeromekelleher deleted the fix-push-up-problem branch October 9, 2026 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants