Skip to content

Set minimum mutations per sample threshold#346

Merged
FerriolCalvet merged 2 commits intodevfrom
min-muts-sample
Aug 26, 2025
Merged

Set minimum mutations per sample threshold#346
FerriolCalvet merged 2 commits intodevfrom
min-muts-sample

Conversation

@FerriolCalvet
Copy link
Collaborator

Samples with no mutations are not analyzed individually

  • same happens with groups that have less than a the chosen number of muts
  • set minimum to 0

- same happens with groups that have less than a the chosen number of muts
- set minimum to 0
- NOT tested
@FerriolCalvet FerriolCalvet requested a review from Copilot August 26, 2025 11:47
@FerriolCalvet FerriolCalvet added this to the Development phase 1 milestone Aug 26, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR introduces a configurable minimum mutations per sample threshold to control when individual sample analysis is performed. It sets a new parameter with a default value of 0 to ensure samples with no mutations are not excluded from analysis.

  • Added configurable min_muts_per_sample parameter with default value of 0
  • Modified mutation subset process to handle optional output when minimum threshold is not met
  • Removed "notcontains p8" from default filter criteria

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.

File Description
nextflow_schema.json Added schema definition for the new min_muts_per_sample parameter
nextflow.config Added min_muts_per_samples parameter and updated filter criteria
modules/local/subsetmaf/main.nf Modified process to accept minimum mutations parameter and made output optional
conf/modules.config Applied minimum mutations parameter to SOMATICMUTATIONS process

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@FerriolCalvet FerriolCalvet merged commit 18523c9 into dev Aug 26, 2025
@FerriolCalvet FerriolCalvet deleted the min-muts-sample branch August 26, 2025 12:08
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