Skip to content

Strelka vep cnaqc patch#48

Merged
valerianilucrezia merged 13 commits intonf-core:devfrom
RaqManzano:strelka-vep-cnaqc-patch
Dec 15, 2025
Merged

Strelka vep cnaqc patch#48
valerianilucrezia merged 13 commits intonf-core:devfrom
RaqManzano:strelka-vep-cnaqc-patch

Conversation

@RaqManzano
Copy link
Contributor

@RaqManzano RaqManzano commented Dec 3, 2025

List of updates

  • Fixes strelka tumor/normal names: in vcf2cnaqc sample names were showing as TUMOR and NORMAL (default in strelka vcf). Now it gets the sample names from metadata.
  • Fixes indels logic (there was an error in previous PR(Added support for strelka indels in VCF2CNAQC #47 )).
  • Removes cnaqc_plot_cn (not present in Rscript template) and added cnaqc_genome to support other genome versions.
  • Changes cnaqc config to the expected format.
  • Small improvement to input staging in JOIN_CNAQC

Copy link
Collaborator

@nicola-calonaci nicola-calonaci left a comment

Choose a reason for hiding this comment

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

Thanks for the feedback! Please see specific comments regarding the genome parameter.

Changing genome param to global genome value
Changing genome param to global genome value
Changing genome param to global genome value
Copy link
Contributor Author

@RaqManzano RaqManzano left a comment

Choose a reason for hiding this comment

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

I removed the param cnaqc_genome in favour of just param genome as requested.

@valerianilucrezia valerianilucrezia merged commit 461ab54 into nf-core:dev Dec 15, 2025
19 of 28 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.

3 participants