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

recalibrate-BAM automatically uses 1000G indels #60

Open
bltsai-UCLA opened this issue Mar 23, 2024 · 3 comments
Open

recalibrate-BAM automatically uses 1000G indels #60

bltsai-UCLA opened this issue Mar 23, 2024 · 3 comments

Comments

@bltsai-UCLA
Copy link

--known-sites Mills_and_1000G_gold_standard.indels.hg38.vcf.gz is automatically added to the recalibrate-BAM run command despite me removing bundle_mills_and_1000g_gold_standard_indels_vcf_gz = "/hot/ref/tool-specific-input/GATK/GRCh38/Mills_and_1000G_gold_standard.indels.hg38.vcf.gz" from the config. We don't want this when aligning to the mouse genome.

config file:
/hot/user/bltsai/projects/JTBRMMEM/metapipeline-DNA/config/recalibrate-BAM.config

working directory:
/hot/user/bltsai/projects/JTBRMMEM/metapipeline-DNA/workdir/44/e9d40b1136141e544b1fc5957e21ff

@zhuchcn
Copy link
Member

zhuchcn commented Mar 24, 2024

Seems like the known sites should be provided as a list of VCF files from the config file instead of hard-coded values of those bundle files

@yashpatel6
Copy link
Collaborator

This is a requirement for the current setup with recalibrate-BAM where the known sites are specifically coded; I'll transfer the issue there

@yashpatel6 yashpatel6 transferred this issue from another repository Mar 25, 2024
@yashpatel6
Copy link
Collaborator

I do agree the sites should be parameterized a bit better to allow for more flexibility; for a temporary solution, I would suggest something like giving an empty VCF for the mouse runs?

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

No branches or pull requests

3 participants