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

Remove unnecessary files from SvABA published files #982

Open
anoronh4 opened this issue Mar 30, 2023 · 2 comments
Open

Remove unnecessary files from SvABA published files #982

anoronh4 opened this issue Mar 30, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@anoronh4
Copy link
Collaborator

publishDir "${params.outDir}/somatic/${idTumor}__${idNormal}/svaba", mode: params.publishDirMode, pattern: "*.{vcf.gz,vcf.gz.tbi}"

Currently, the publishDir directive in SomaticRunSvABA and GermlineRunSvABA both output all vcf files found in the work directory.
SvABA generates several vcf files including indel calls which we might consider not publishing or not generating at all if we can help it, because we don't use them and they aren't being added to the maf.
The process SomaticRunSvABA and GermlineRunSvABA also produce vcfs file with a slightly modified header, which helps when merging the SV calls in subsequent steps. I think we should also not publish these files, as there is nothing different in the calls compared to the original vcf.

@anoronh4 anoronh4 added the bug Something isn't working label Mar 30, 2023
@johnoooh
Copy link
Collaborator

I'm using the SVABA indels for consensus calling so I wouldn't change them for now.

@anoronh4
Copy link
Collaborator Author

anoronh4 commented Apr 7, 2023

then maybe we should make this configurable? I wonder if it will confuse endUsers if they see the svaba indel files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants