TSPS-183 Beagle imputation hg38 wdl and associated support wdls#1333
TSPS-183 Beagle imputation hg38 wdl and associated support wdls#1333nikellepetrillo merged 96 commits intodevelopfrom
Conversation
|
Can one of the admins verify this patch? |
pullapprove
bot
requested review from
fleharty,
gbggrant,
kachulis and
samuelklee
|
Remember to squash merge! |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
Remember to squash merge! |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
Remember to squash merge! |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
@mmorgantaylor is this still relevant? |
|
@jessicaway yes this is our working branch for beagle imputation for the imputation service. cc @jsotobroad. Jess, would it be helpful for us to squash and rebase off of develop? |
|
OK, no problem! Whatever works best is fine. We were just cleaning up PRs and thought we would check in. I don't think we have touched imputation much recently, so I don't anticipate any conflicts |
mmorgantaylor
force-pushed
the
TSPS-183_mma_beagle_imputation_hg38
branch
from
79d3510 to
eaeabdc
Compare
|
Remember to squash merge! |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
mmorgantaylor
force-pushed
the
TSPS-183_mma_beagle_imputation_hg38
branch
from
4f107d0 to
0a302fe
Compare
|
Remember to squash merge! |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
mmorgantaylor
force-pushed
the
TSPS-183_mma_beagle_imputation_hg38
branch
from
0a302fe to
6eb5bfc
Compare
|
Remember to squash merge! |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
🔍Changelog Validation Results: |
|
Remember to squash merge! |
🔍Version Validation Results: |
🔍Changelog Validation Results: |
| # 1.0.0 | ||
| 2025-02-24 (Date of Last Commit) | ||
|
|
||
| * * Initial public release of the ImputationBeagle pipeline. |
There was a problem hiding this comment.
| * * Initial public release of the ImputationBeagle pipeline. | |
| * Initial public release of the ImputationBeagle pipeline. |
| 2025-02-24 (Date of Last Commit) | ||
|
|
||
| * * Initial public release of the ImputationBeagle pipeline. | ||
|
|
| @@ -0,0 +1,8 @@ | |||
| { | |||
| "ImputationBeagle.multi_sample_vcf": "gs://broad-gotc-test-storage/imputation_beagle/scientific/vcfs/NA12878_10_duplicate.merged.cleaned.vcf.gz", | |||
There was a problem hiding this comment.
i see that this path points to a vcf.gz in a scientific directory, is that expected?
There was a problem hiding this comment.
it is - the way the scientific test differs from plumbing is that scientific runs on all chromosomes, and plumbing only runs on on a couple smaller chromosomes. the input data files themselves are the same. is that acceptable? i think the plumbing runs in just over an hour.
| File ref_panel_bref3 | ||
| File genetic_map_file | ||
| String basename | ||
| String chrom # not needed if ref file has been chunked and you are using the entire chunk |
There was a problem hiding this comment.
if these are not needed, should they be optional inputs?
There was a problem hiding this comment.
ah these were comments from our collaborator when we were creating this WDL; they are indeed needed. I'll remove these comments since they're confusing.
|
Remember to squash merge! |
🔍Version Validation Results: |
🔍Changelog Validation Results: |
|
Remember to squash merge! |
🔍Changelog Validation Results: |
🔍Version Validation Results: |
|
Remember to squash merge! |
🔍Version Validation Results: |
🔍Changelog Validation Results: |
|
Remember to squash merge! |
🔍Changelog Validation Results: |
🔍Version Validation Results: |
7 participants
Description
We are adding a new WDL, ImputationBeagle.wdl, that performs array imputation using the Beagle tool instead of minimac4, which is used by the existing Imputation.wdl.
Supporting PR in warp-tools for the Beagle docker: broadinstitute/warp-tools#114
Jira ticket: https://broadworkbench.atlassian.net/browse/TSPS-222
Checklist
If you can answer "yes" to the following items, please add a checkmark next to the appropriate checklist item(s) and notify our WARP documentation team by tagging either @ekiernan or @kayleemathews in a comment on this PR.