Skip to content

Add estimator flag to include zero-mutation groups#56

Draft
FerriolCalvet with Copilot wants to merge 3 commits into
main-devfrom
copilot/add-option-ignore-dn-ds-computation
Draft

Add estimator flag to include zero-mutation groups#56
FerriolCalvet with Copilot wants to merge 3 commits into
main-devfrom
copilot/add-option-ignore-dn-ds-computation

Conversation

Copilot AI commented May 19, 2026

Copy link
Copy Markdown
Contributor

Estimator currently skips groups with 0 observed mutations, preventing omega values for those cases. This adds a CLI option to control whether those zero-mutation groups are ignored or included.

  • CLI surface

    • Adds a paired flag to toggle zero-mutation handling for the estimator.
  • Estimator data flow

    • Threads the flag through estimator preparation and assembler input generation to conditionally skip or keep zero-mutation groups.
  • Documentation

    • Notes the new flag in the estimator options section.

Example usage:

omega estimator \
  --observed-mutations-file observed.tsv \
  --mutability-file mutability.tsv \
  --depths-file depths.tsv \
  --vep-annotation-file vep.tsv \
  --grouping-folder grouping/ \
  --output-fn results.tsv \
  --include-zero-mutations

Agent-Logs-Url: https://github.com/bbglab/omega/sessions/28ec0e0c-5acf-4c3d-89b4-4d1789806133

Co-authored-by: FerriolCalvet <38539786+FerriolCalvet@users.noreply.github.com>
Copilot AI changed the title [WIP] Add option to not ignore dN/dS computation for cases with 0 mutations Add estimator flag to include zero-mutation groups May 19, 2026
Copilot AI requested a review from FerriolCalvet May 19, 2026 20:49
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.

Add option to not ignore dN/dS computation for cases with 0 mutations

2 participants