Skip to content

Fix AHO phase parity for numeric-only GT handling#116

Merged
Jaureguy760 merged 1 commit into
mcvickerlab:devfrom
Jaureguy760:dev
Jun 5, 2026
Merged

Fix AHO phase parity for numeric-only GT handling#116
Jaureguy760 merged 1 commit into
mcvickerlab:devfrom
Jaureguy760:dev

Conversation

@Jaureguy760

Copy link
Copy Markdown
Collaborator

Summary

  • restrict analyze_imbalance GT phase parsing to numeric VCF-style phased hets (0|1 and 1|0)
  • avoid deriving haplotype phase from allele-string GT values like A|G
  • fall back to the unphased model when requested phased mode lacks informative numeric phase

Verification

  • cargo fmt --check
  • git diff --check -- src/lib.rs src/analysis.rs
  • cargo test --lib: 116 passed; 0 failed; 7 ignored

Benchmark context

This is the Rust candidate commit for the fresh AHO-vs-Rust phase parity lock. AHO remains the ground truth; donor dispersion remains a Rust-only extension and is not treated as AHO parity.

@Jaureguy760 Jaureguy760 merged commit 541624d into mcvickerlab:dev Jun 5, 2026
8 checks passed
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.

1 participant