Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: fusion calling #222

Merged
merged 68 commits into from
Feb 27, 2024
Merged

feat: fusion calling #222

merged 68 commits into from
Feb 27, 2024

Conversation

FelixMoelder
Copy link
Contributor

@FelixMoelder FelixMoelder commented Mar 16, 2023

These changes introduce fusion calling in addition the variant calling.
There for arriba is applied for calling variants. As vep does not provide any helpful information it is skipped for fusion calls.
Instead annotation is performed by additional scripts that come with the arriba package.
To define a what kind of analysis shall be performed for a sample an additional column named datatype has been added to the samplesheet allowing either rna or dna. As file name extensions within the pipeline are separated into variants and fusions we probably should unify this as the former one might also be too generic.
For fusion calls a separte datavzrd report will be created adding a new category showing up in the final snakemake report.

@FelixMoelder
Copy link
Contributor Author

To get this running we need suhrig/arriba#191 being merged and a new release of arriba.

@FelixMoelder FelixMoelder marked this pull request as ready for review September 20, 2023 13:49
config/config.yaml Outdated Show resolved Hide resolved
config/config.yaml Show resolved Hide resolved
workflow/report/workflow.rst Outdated Show resolved Hide resolved
workflow/rules/datavzrd.smk Outdated Show resolved Hide resolved
workflow/rules/datavzrd.smk Outdated Show resolved Hide resolved
workflow/rules/datavzrd.smk Outdated Show resolved Hide resolved
@@ -0,0 +1,90 @@
#!/bin/bash
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this script come from arriba sources? Please cite the full URL here.

Copy link
Contributor

Choose a reason for hiding this comment

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

And is the license compatible?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The license of arriba is also MIT and therefore compatible. :)

@johanneskoester johanneskoester enabled auto-merge (squash) February 26, 2024 20:27
@johanneskoester johanneskoester merged commit d0b45ba into master Feb 27, 2024
7 checks passed
@johanneskoester johanneskoester deleted the fusion_calling branch February 27, 2024 06:17
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