Merge pull request icgc-argo-workflows#9 from icgc-argo/sanger-wgs-va…

…riant-calling@2.1.0-9.4.0 [release]
icgc-argo · Apr 16, 2021 · e2f9eb8 · e2f9eb8
2 parents 1e95adb + 433ae3d
commit e2f9eb8
Show file tree

Hide file tree

Showing 10 changed files with 449 additions and 12 deletions.
diff --git a/....0/tools/prep-sanger-qc/prep-sanger-qc.nf → ....0/tools/prep-sanger-qc/prep-sanger-qc.nf b/....0/tools/prep-sanger-qc/prep-sanger-qc.nf → ....0/tools/prep-sanger-qc/prep-sanger-qc.nf
@@ -23,16 +23,20 @@
  */
 
 nextflow.preview.dsl = 2
-version = '0.1.2.0'
+version = '0.1.3.0'
 
 params.qc_files = ""
+params.publish_dir = ""
+
 params.container_version = ""
 params.cpus = 1
 params.mem = 2  // in GB
 
 
 process prepSangerQc {
   container "quay.io/icgc-argo/prep-sanger-qc:prep-sanger-qc.${params.container_version ?: version}"
+  publishDir "${params.publish_dir}/${task.process.replaceAll(':', '_')}", mode: "copy", enabled: params.publish_dir
+
   cpus params.cpus
   memory "${params.mem} GB"
 

diff --git a/sanger-wgs-variant-calling/main.nf b/sanger-wgs-variant-calling/main.nf
@@ -2,7 +2,7 @@
 nextflow.preview.dsl=2
 name='sanger-wgs-variant-calling'
 short_name='sanger-wgs'
-version = '2.1.0-9.3.0'
+version = '2.1.0-9.4.0'
 
 /*
 ========================================================================================
@@ -276,9 +276,9 @@ include { sangerWgsVariantCaller as sangerWgs } from './modules/raw.githubuserco
 include { repackSangerResults as repack } from './modules/raw.githubusercontent.com/icgc-argo/variant-calling-tools/repack-sanger-results.0.2.0.0/tools/repack-sanger-results/repack-sanger-results' params(repackSangerResults_params)
 include { cavemanVcfFix as cavemanFix } from './modules/raw.githubusercontent.com/icgc-argo/variant-calling-tools/caveman-vcf-fix.0.1.0.0/tools/caveman-vcf-fix/caveman-vcf-fix' params(cavemanVcfFix_params)
 include { prepSangerSupplement as prepSupp } from './modules/raw.githubusercontent.com/icgc-argo/variant-calling-tools/prep-sanger-supplement.0.1.2.0/tools/prep-sanger-supplement/prep-sanger-supplement' params(prepSangerSupplement_params)
-include { prepSangerQc as prepQc } from './modules/raw.githubusercontent.com/icgc-argo/variant-calling-tools/prep-sanger-qc.0.1.2.0/tools/prep-sanger-qc/prep-sanger-qc' params(prepSangerQc_params)
+include { prepSangerQc as prepQc } from './modules/raw.githubusercontent.com/icgc-argo/variant-calling-tools/prep-sanger-qc.0.1.3.0/tools/prep-sanger-qc/prep-sanger-qc' params(prepSangerQc_params)
 include { extractFilesFromTarball as extractVarSnv; extractFilesFromTarball as extractVarIndel; extractFilesFromTarball as extractVarCnv; extractFilesFromTarball as extractVarSv } from './modules/raw.githubusercontent.com/icgc-argo/data-processing-utility-tools/extract-files-from-tarball.0.2.0.0/tools/extract-files-from-tarball/extract-files-from-tarball' params(extractSangerCall_params)
-include { payloadGenVariantCalling as pGenVarSnv; payloadGenVariantCalling as pGenVarIndel; payloadGenVariantCalling as pGenVarCnv; payloadGenVariantCalling as pGenVarSv; payloadGenVariantCalling as pGenVarSupp; payloadGenVariantCalling as pGenQc } from "./modules/raw.githubusercontent.com/icgc-argo/data-processing-utility-tools/payload-gen-variant-calling.0.3.6.0/tools/payload-gen-variant-calling/payload-gen-variant-calling" params(payloadGenVariantCall_params)
+include { payloadGenVariantCalling as pGenVarSnv; payloadGenVariantCalling as pGenVarIndel; payloadGenVariantCalling as pGenVarCnv; payloadGenVariantCalling as pGenVarSv; payloadGenVariantCalling as pGenVarSupp; payloadGenVariantCalling as pGenQc } from "./wfpr_modules/github.com/icgc-argo/data-processing-utility-tools/payload-gen-variant-calling@0.4.0/main.nf" params(payloadGenVariantCall_params)
 include { SongScoreUpload as upSnv; SongScoreUpload as upIndel; SongScoreUpload as upCnv; SongScoreUpload as upSv; SongScoreUpload as upQc; SongScoreUpload as upSupp} from './wfpr_modules/github.com/icgc-argo/nextflow-data-processing-utility-tools/song-score-upload@2.6.1/main.nf' params(upload_params)
 include { cleanupWorkdir as cleanup } from './wfpr_modules/github.com/icgc-argo/data-processing-utility-tools/cleanup-workdir@1.0.0/main'
 include { getSecondaryFiles } from './wfpr_modules/github.com/icgc-argo/data-processing-utility-tools/helper-functions@1.0.0/main'

diff --git a/sanger-wgs-variant-calling/pkg.json b/sanger-wgs-variant-calling/pkg.json
@@ -1,6 +1,6 @@
 {
     "name": "sanger-wgs-variant-calling",
-    "version": "2.1.0-9.3.0",
+    "version": "2.1.0-9.4.0",
     "description": "ICGC ARGO Sanger WGS Variant Calling Workflow",
     "main": "main.nf",
     "deprecated": false,
@@ -15,6 +15,7 @@
         "url": "https://github.com/icgc-argo/sanger-wgs-variant-calling.git"
     },
     "dependencies": [
+        "github.com/icgc-argo/data-processing-utility-tools/payload-gen-variant-calling@0.4.0",
         "github.com/icgc-argo/data-processing-utility-tools/payload-add-uniform-ids@0.1.1",
         "github.com/icgc-argo/data-processing-utility-tools/helper-functions@1.0.0",
         "github.com/icgc-argo/data-processing-utility-tools/cleanup-workdir@1.0.0",

diff --git a/...m/icgc-argo/data-processing-utility-tools/payload-gen-variant-calling@0.4.0/.dockerignore b/...m/icgc-argo/data-processing-utility-tools/payload-gen-variant-calling@0.4.0/.dockerignore
@@ -0,0 +1,5 @@
+.gitignore
+.nextflow*
+tests
+work
+outdir
diff --git a/....com/icgc-argo/data-processing-utility-tools/payload-gen-variant-calling@0.4.0/Dockerfile b/....com/icgc-argo/data-processing-utility-tools/payload-gen-variant-calling@0.4.0/Dockerfile
@@ -0,0 +1,18 @@
+FROM python:3.7.9
+
+LABEL org.opencontainers.image.source https://github.com/icgc-argo/data-processing-utility-tools
+
+RUN groupadd -g 1000 ubuntu && \
+    useradd -l -u 1000 -g ubuntu ubuntu && \
+    install -d -m 0755 -o ubuntu -g ubuntu /home/ubuntu
+
+ENV PATH="/tools:${PATH}"
+
+COPY *.py /tools/
+
+WORKDIR /tools
+
+USER ubuntu
+
+ENTRYPOINT ["/usr/bin/env"]
+CMD ["/bin/bash"]
diff --git a/...nt-calling/payload-gen-variant-calling.nf → ...payload-gen-variant-calling@0.4.0/main.nf b/...nt-calling/payload-gen-variant-calling.nf → ...payload-gen-variant-calling@0.4.0/main.nf
@@ -21,22 +21,39 @@
  * Author Junjun Zhang <junjun.zhang@oicr.on.ca>
  */
 
+/********************************************************************/
+/* this block is auto-generated based on info from pkg.json where   */
+/* changes can be made if needed, do NOT modify this block manually */
 nextflow.enable.dsl = 2
-version = '0.3.6.0'
+version = '0.4.0'  // package version
 
+container = [
+    'ghcr.io': 'ghcr.io/icgc-argo/data-processing-utility-tools.payload-gen-variant-calling'
+]
+default_container_registry = 'ghcr.io'
+/********************************************************************/
+
+
+// universal params go here
+params.container_registry = ""
+params.container_version = ""
+params.container = ""
+
+params.cpus = 1
+params.mem = 1  // GB
+params.publish_dir = ""  // set to empty string will disable publishDir
+
+// tool specific parmas go here, add / change as needed
 params.normal_analysis = ""
 params.tumour_analysis = ""
 params.files_to_upload = []
 params.wf_name = ""
 params.wf_short_name = ""
 params.wf_version = ""
-params.container_version = ''
-params.cpus = 1
-params.mem = 1  // GB
-params.publish_dir = ""
+
 
 process payloadGenVariantCalling {
-  container "quay.io/icgc-argo/payload-gen-variant-calling:payload-gen-variant-calling.${params.container_version ?: version}"
+  container "${params.container ?: container[params.container_registry ?: default_container_registry]}:${params.container_version ?: version}"
   cpus params.cpus
   memory "${params.mem} GB"
   publishDir "${params.publish_dir}/${task.process.replaceAll(':', '_')}", mode: "copy", enabled: "${params.publish_dir ? true : ''}"
@@ -56,7 +73,7 @@ process payloadGenVariantCalling {
   script:
     args_tumour_analysis = !tumour_analysis.empty() ? "-t ${tumour_analysis}" : ""
     """
-    payload-gen-variant-calling.py \
+    main.py \
          -f ${files_to_upload} \
          -n ${normal_analysis} \
          -r ${workflow.runName} \
@@ -66,3 +83,16 @@ process payloadGenVariantCalling {
          -v ${wf_version} ${args_tumour_analysis}
     """
 }
+
+// this provides an entry point for this main script, so it can be run directly without clone the repo
+// using this command: nextflow run <git_acc>/<repo>/<pkg_name>/<main_script>.nf -r <pkg_name>.v<pkg_version> --params-file xxx
+workflow {
+  payloadGenVariantCalling(
+    file(params.normal_analysis),
+    file(params.tumour_analysis),
+    Channel.fromPath(params.files_to_upload).collect(),
+    params.wf_name,
+    params.wf_short_name,
+    params.wf_version
+  )
+}