-
Notifications
You must be signed in to change notification settings - Fork 592
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
Added code and WDL to complete ModelSegments CNV pipeline. #3913
Changes from all commits
811770f
5d08c52
d39e6ea
07f2526
c6bd294
9b08506
f9faeab
e2f5275
2371b43
c94e762
05500d1
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -13,6 +13,7 @@ env: | |
- TEST_TYPE=integration TEST_DOCKER=true TEST_VERBOSITY=minimal | ||
- TEST_TYPE=unit TEST_DOCKER=true TEST_VERBOSITY=minimal | ||
- TEST_TYPE=python TEST_DOCKER=true TEST_VERBOSITY=minimal | ||
- RUN_CNV_SOMATIC_WDL=true | ||
- RUN_CNV_SOMATIC_LEGACY_WDL=true | ||
- RUN_M2_WDL=true | ||
global: | ||
|
@@ -87,7 +88,7 @@ before_install: | |
sudo Rscript scripts/docker/gatkbase/install_R_packages.R; | ||
fi | ||
# Download Cromwell jar | ||
- if [[ $RUN_CNV_SOMATIC_LEGACY_WDL == true || $RUN_M2_WDL == true || $RUN_CNV_GERMLINE_WDL == true ]]; then | ||
- if [[ $RUN_CNV_SOMATIC_LEGACY_WDL == true || $RUN_CNV_SOMATIC_WDL == true || $RUN_CNV_GERMLINE_WDL == true || $RUN_M2_WDL == true ]]; then | ||
wget -O ~/cromwell-0.28.jar https://github.com/broadinstitute/cromwell/releases/download/28/cromwell-28.jar; | ||
fi | ||
# Download Picard jar | ||
|
@@ -107,7 +108,7 @@ install: | |
else | ||
./gradlew assemble; | ||
./gradlew installDist; | ||
if [[ $RUN_CNV_SOMATIC_LEGACY_WDL == true || $RUN_M2_WDL == true || $RUN_CNV_GERMLINE_WDL == true ]]; then | ||
if [[ $RUN_CNV_SOMATIC_LEGACY_WDL == true || $RUN_CNV_SOMATIC_WDL == true || $RUN_CNV_GERMLINE_WDL == true || $RUN_M2_WDL == true ]]; then | ||
echo "building a shadow jar for the wdl"; | ||
./gradlew shadowJar; | ||
elif [[ $TEST_TYPE == cloud ]]; then | ||
|
@@ -122,12 +123,12 @@ script: | |
echo "Not running any tests for nightly builds"; | ||
elif [[ $TRAVIS_SECURE_ENV_VARS == false && $TEST_TYPE == cloud ]]; then | ||
echo "Can't run cloud tests without keys so don't run tests"; | ||
elif [[ $RUN_CNV_SOMATIC_WDL == true ]]; then | ||
echo "Running CNV somatic workflows"; | ||
bash scripts/cnv_cromwell_tests/somatic/run_cnv_somatic_workflows.sh; | ||
elif [[ $RUN_CNV_SOMATIC_LEGACY_WDL == true ]]; then | ||
echo "Running legacy CNV somatic workflows"; | ||
bash scripts/cnv_cromwell_tests/somatic_legacy/run_cnv_somatic_workflows.sh; | ||
elif [[ $RUN_CNV_GERMLINE_WDL == true ]]; then | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why are we no longer running germline tests? Is it because we are going to roll out the new workflow and don't want the support burden? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Also cleaned up in the germline PRs. The new workflows are run there. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. FYI, gCNV Spark also got blown away. |
||
echo "Running CNV germline workflows"; | ||
bash scripts/cnv_cromwell_tests/germline/run_cnv_germline_workflows.sh; | ||
elif [[ $RUN_M2_WDL == true ]]; then | ||
echo "Deleting some unused files before running M2 WDL..."; | ||
rm -Rf src/test/resources/large/VQSR; | ||
|
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're still going to run the legacy WDL? Haven't you removed the code in this PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is cleaned up in a later PR.