diff --git a/protected/biopet-gatk-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/gatk/broad/GenotypeGVCFs.scala b/protected/biopet-gatk-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/gatk/broad/GenotypeGVCFs.scala index b7d68456c3b601728cb6b637b0690c761d6b0726..30ec0d09721ee7d06c69da3fffa3681579b7cb19 100644 --- a/protected/biopet-gatk-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/gatk/broad/GenotypeGVCFs.scala +++ b/protected/biopet-gatk-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/gatk/broad/GenotypeGVCFs.scala @@ -8,7 +8,7 @@ package nl.lumc.sasc.biopet.extensions.gatk.broad import java.io.File import nl.lumc.sasc.biopet.utils.config.Configurable -import org.broadinstitute.gatk.utils.commandline.{Gather, Output} +import org.broadinstitute.gatk.utils.commandline.{ Gather, Output } class GenotypeGVCFs(val root: Configurable) extends org.broadinstitute.gatk.queue.extensions.gatk.GenotypeGVCFs with GatkGeneral { diff --git a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/bcftools/BcftoolsView.scala b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/bcftools/BcftoolsView.scala index 681c966567469cd7da431608d594aa4a129cb7ad..ff3ed0ecb94003d6e4076a32686ec97ec82f5948 100644 --- a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/bcftools/BcftoolsView.scala +++ b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/bcftools/BcftoolsView.scala @@ -17,10 +17,10 @@ class BcftoolsView(val root: Configurable) extends Bcftools { var output: File = _ @Argument(doc = "drop individual genotypes", required = false) - var dropGenotype: Boolean = false + var dropGenotype: Boolean = config("drop_genotype", default = false) @Argument(doc = "header only", required = false) - var headerOnly: Boolean = false + var headerOnly: Boolean = config("header_only", false) @Argument(doc = "Compression level", required = false) var compressionLevel: Int = 9 @@ -29,88 +29,88 @@ class BcftoolsView(val root: Configurable) extends Bcftools { var outputType: String = "z" @Argument(doc = "regions", required = false) - var regions: String = _ + var regions: Option[String] = config("r") @Argument(doc = "region file", required = false) - var regionFile: File = _ + var regionFile: Option[File] = config("R") @Argument(doc = "targets", required = false) - var targets: String = _ + var targets: Option[String] = config("t") @Argument(doc = "targets file", required = false) - var targetFile: File = _ + var targetFile: Option[File] = config("T") @Argument(doc = "trim alt alleles", required = false) - var trimAltAlleles: Boolean = false + var trimAltAlleles: Boolean = config("trim_alt_allele", default = false) @Argument(doc = "no update", required = false) - var noUpdate: Boolean = false + var noUpdate: Boolean = config("no_update", default = false) @Argument(doc = "samples", required = false) - var samples: List[String] = Nil + var samples: List[String] = config("s", default = Nil) @Argument(doc = "samples file", required = false) - var sampleFile: File = _ + var sampleFile: Option[File] = config("S") @Argument(doc = "minimum allele count", required = false) - var minAC: Option[Int] = _ + var minAC: Option[Int] = config("c") @Argument(doc = "max allele count", required = false) - var maxAC: Option[Int] = _ + var maxAC: Option[Int] = config("C") @Argument(doc = "exclude (expression)", required = false) - var exclude: String = _ + var exclude: Option[String] = config("e") @Argument(doc = "apply filters", required = false) - var applyFilters: List[String] = Nil + var applyFilters: List[String] = config("F", default = Nil) @Argument(doc = "genotype", required = false) - var genotype: String = _ + var genotype: Option[String] = config("g") @Argument(doc = "include (expression)", required = false) - var include: String = _ + var include: Option[String] = config("i") @Argument(doc = "Known (ID field is not .) only", required = false) - var known: Boolean = false + var known: Boolean = config("k", default = false) @Argument(doc = "min alleles", required = false) - var minAlleles: Option[Int] = _ + var minAlleles: Option[Int] = config("m") @Argument(doc = "max alleles", required = false) - var maxAlleles: Option[Int] = _ + var maxAlleles: Option[Int] = config("M") @Argument(doc = "novel (ID field is .) only", required = false) - var novel: Boolean = false + var novel: Boolean = config("n", false) @Argument(doc = "phased only", required = false) - var phased: Boolean = false + var phased: Boolean = config("p", false) @Argument(doc = "exclude phased (only)", required = false) - var excludePhased: Boolean = false + var excludePhased: Boolean = config("P", false) @Argument(doc = "min allele frequency", required = false) - var minAF: Option[Int] = _ + var minAF: Option[Int] = config("q") @Argument(doc = "max allele frequency", required = false) - var maxAF: Option[Int] = _ + var maxAF: Option[Int] = config("Q") @Argument(doc = "uncalled only", required = false) - var uncalled: Boolean = false + var uncalled: Boolean = config("u", default = false) @Argument(doc = "exclude uncalled (only)", required = false) - var excludeUncalled: Boolean = false + var excludeUncalled: Boolean = config("U", default = false) @Argument(doc = "types", required = false) - var types: String = _ + var types: Option[String] = config("v") @Argument(doc = "exclude types", required = false) - var excludeTypes: String = _ + var excludeTypes: Option[String] = config("V") @Argument(doc = "private (requires samples)", required = false) - var onlyPrivate: Boolean = false + var onlyPrivate: Boolean = config("x", default = false) @Argument(doc = "Exclude privates", required = false) - var excludePrivate: Boolean = false + var excludePrivate: Boolean = config("X", default = false) override def beforeGraph() = { super.beforeGraph() @@ -123,21 +123,39 @@ class BcftoolsView(val root: Configurable) extends Bcftools { } def baseCmd = { - executable + " view " + conditional(dropGenotype, "-G") + conditional(headerOnly, "-h") + - required("-l", compressionLevel) + required("-O", outputType) + - optional("-r", regions) + optional("-R", regionFile) + - optional("-t", targets) + optional("-T", targetFile) + - conditional(trimAltAlleles, "-a") + conditional(noUpdate, "-I") + - repeat("-s", samples) + optional("-S", sampleFile) + - optional("-c", minAC) + optional("-C", maxAC) + - optional("-e", exclude) + optional("-f", applyFilters) + - optional("-g", genotype) + optional("-i", include) + - conditional(known, "-k") + optional("-m", minAlleles) + - optional("-M", maxAlleles) + conditional(novel, "-n") + - conditional(phased, "-p") + conditional(excludePhased, "-P") + - optional("-q", minAF) + optional("-Q", maxAF) + - conditional(uncalled, "-u") + conditional(excludeUncalled, "-U") + - optional("-v", types) + conditional(onlyPrivate, "-x") + + executable + + required("view") + + conditional(dropGenotype, "-G") + + conditional(headerOnly, "-h") + + required("-l", compressionLevel) + + required("-O", outputType) + + optional("-r", regions) + + optional("-R", regionFile) + + optional("-t", targets) + + optional("-T", targetFile) + + conditional(trimAltAlleles, "-a") + + conditional(noUpdate, "-I") + + repeat("-s", samples) + + optional("-S", sampleFile) + + optional("-c", minAC) + + optional("-C", maxAC) + + optional("-e", exclude) + + optional("-f", applyFilters) + + optional("-g", genotype) + + optional("-i", include) + + conditional(known, "-k") + + optional("-m", minAlleles) + + optional("-M", maxAlleles) + + conditional(novel, "-n") + + conditional(phased, "-p") + + conditional(excludePhased, "-P") + + optional("-q", minAF) + + optional("-Q", maxAF) + + conditional(uncalled, "-u") + + conditional(excludeUncalled, "-U") + + optional("-v", types) + + optional("-V", excludeTypes) + + conditional(onlyPrivate, "-x") + conditional(excludePrivate, "-X") } diff --git a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/bedtools/BedtoolsMerge.scala b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/bedtools/BedtoolsMerge.scala index c0f2e36407c3ef0c7a4fac20a9e087279fb6832c..f72411a6ab7b83c7661d4f36ec38d12bca3136c1 100644 --- a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/bedtools/BedtoolsMerge.scala +++ b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/bedtools/BedtoolsMerge.scala @@ -14,7 +14,7 @@ class BedtoolsMerge(val root: Configurable) extends Bedtools { var input: File = _ @Argument(doc = "Distance") - var dist: Int = 1 //default of tool is 1 + var dist: Option[Int] = config("dist") //default of tool is 1 @Output(doc = "Output bed file") var output: File = _ diff --git a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/Manwe.scala b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/Manwe.scala index b352a998f1056e00693e6f389dd31ac1052693eb..e2566088d254a9e467d1a00a90e477e431c6d534 100644 --- a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/Manwe.scala +++ b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/Manwe.scala @@ -13,9 +13,6 @@ import org.broadinstitute.gatk.utils.commandline.{ Output, Argument } abstract class Manwe extends BiopetCommandLineFunction { executable = config("exe", default = "manwe", submodule = "manwe") - override def defaultCoreMemory = 2.0 - override def defaultThreads = 1 - var manweConfig: File = createManweConfig(None) @Output(doc = "the output file") @@ -58,31 +55,28 @@ abstract class Manwe extends BiopetCommandLineFunction { val dataBufferSize: Option[Int] = config("varda_buffer_size", default = 1024 * 1024) val taskPollWait: Option[Int] = config("varda_task_poll_wait", default = 2) - val urlString = s"API_ROOT = '${url.toString}'" - val tokenString = s"TOKEN = '${token.toString}'" - val sslSettingString = sslSettings match { - case Some("true") => "VERIFY_CERTIFICATE = True" - case Some("false") => "VERIFY_CERTIFICATE = False" - case Some(x) => s"VERIFY_CERTIFICATE = '$x'" - case _ => "VERIFY_CERTIFICATE = True" - } + val settingsMap: Map[String, Any] = Map( + "API_ROOT" -> s"'$url'", + "TOKEN" -> s"'$token'", + "VERIFY_CERTIFICATE" -> (sslSettings match { + case Some("true") => "True" + case Some("false") => "False" + case Some(x) => s"'$x'" + case _ => "True" + }), + "COLLECTION_CACHE_SIZE" -> collectionCacheSize.getOrElse(20), + "DATA_BUFFER_SIZE" -> dataBufferSize.getOrElse(1048576), + "TASK_POLL_WAIT" -> taskPollWait.getOrElse(2) + ) - val collectionString = s"COLLECTION_CACHE_SIZE = ${collectionCacheSize.getOrElse(20)}" - val dataString = s"DATA_BUFFER_SIZE = ${dataBufferSize.getOrElse(1048576)}" - val taskString = s"TASK_POLL_WAIT = ${taskPollWait.getOrElse(2)}" val file = directory match { case Some(dir) => File.createTempFile("manwe_config", ".py", dir) case None => File.createTempFile("manwe_config", ".py") } + file.deleteOnExit() val writer = new PrintWriter(file) - writer.println(urlString) - writer.println(tokenString) - writer.println(sslSettingString) - writer.println(collectionString) - writer.println(dataString) - writer.println(taskString) - + settingsMap.foreach { case (key, value) => writer.println(s"$key = $value") } writer.close() file } diff --git a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweAnnotateBed.scala b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweAnnotateBed.scala index 752b0fa7ecf8148ae8c3d40a4d289919fbe444eb..f3c3fbb1e22a019a18b7eb8caa6d262e2f3347f8 100644 --- a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweAnnotateBed.scala +++ b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweAnnotateBed.scala @@ -25,7 +25,8 @@ class ManweAnnotateBed(val root: Configurable) extends Manwe { def subCommand = { required("annotate-bed") + required(bed) + conditional(alreadyUploaded, "-u") + - repeat("-q", queries) + conditional(waitToComplete, "--wait") + repeat("-q", queries) + + conditional(waitToComplete, "--wait") } } diff --git a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweAnnotateVcf.scala b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweAnnotateVcf.scala index e469d65e541bb5710d1d41fddcf1f831a1cdd756..64a849a536d4695facbfd290379426fd3d646287 100644 --- a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweAnnotateVcf.scala +++ b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweAnnotateVcf.scala @@ -25,7 +25,8 @@ class ManweAnnotateVcf(val root: Configurable) extends Manwe { def subCommand = { required("annotate-vcf") + required(vcf) + conditional(alreadyUploaded, "-u") + - repeat("-q", queries) + conditional(waitToComplete, "--wait") + repeat("-q", queries) + + conditional(waitToComplete, "--wait") } } diff --git a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweDataSourcesAnnotate.scala b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweDataSourcesAnnotate.scala index 682fedba50256b2a1683ad5119ac45fe34b43456..0980b4c23d4450c95b17baa8c1d2f52bfb6a63d5 100644 --- a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweDataSourcesAnnotate.scala +++ b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweDataSourcesAnnotate.scala @@ -19,7 +19,8 @@ class ManweDataSourcesAnnotate(val root: Configurable) extends Manwe { def subCommand = { required("data-sources") + required("annotate") + - required(uri) + repeat("-q", queries) + + required(uri) + + repeat("-q", queries) + conditional(waitToComplete, "--wait") } diff --git a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweDataSourcesDownload.scala b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweDataSourcesDownload.scala index e239564c887e532902ae49e1478a563fcc924dd1..d55dd6a4f8c8c181bd565c60d085b938a08fe4a0 100644 --- a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweDataSourcesDownload.scala +++ b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweDataSourcesDownload.scala @@ -12,9 +12,8 @@ class ManweDataSourcesDownload(val root: Configurable) extends Manwe { var uri: String = _ def subCommand = { - required("data-sources") + required("download") + required(uri) + required("data-sources") + + required("download") + + required(uri) } - - this.deps - } diff --git a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweDataSourcesList.scala b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweDataSourcesList.scala index 0e2408a3c23969021e24a3f20cdc03f9553864e7..dbcb7b46913e7bb66ba41d74bc141e3ed861a0b0 100644 --- a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweDataSourcesList.scala +++ b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweDataSourcesList.scala @@ -14,7 +14,8 @@ class ManweDataSourcesList(val root: Configurable) extends Manwe { var user: Option[String] = _ def subCommand = { - required("data-sources") + required("list") + + required("data-sources") + + required("list") + optional("-u", user) } diff --git a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweDataSourcesShow.scala b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweDataSourcesShow.scala index f2d108e2d21662c4b0d6e9f84ebb03f6508bd498..662bb51b79e292a352d524ca2437dd953737d894 100644 --- a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweDataSourcesShow.scala +++ b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweDataSourcesShow.scala @@ -14,7 +14,8 @@ class ManweDataSourcesShow(val root: Configurable) extends Manwe { var uri: Option[String] = _ def subCommand = { - required("data-sources") + required("show") + + required("data-sources") + + required("show") + required(uri) } diff --git a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesActivate.scala b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesActivate.scala index 4ddd3dfb54b859409e137e11475fb18cb1c8c499..3815d64a258dd52b354bb2be2be1610d6eb2d4cb 100644 --- a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesActivate.scala +++ b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesActivate.scala @@ -14,7 +14,8 @@ class ManweSamplesActivate(val root: Configurable) extends Manwe { var uri: String = _ def subCommand = { - required("samples") + required("activate") + + required("samples") + + required("activate") + required(uri) } diff --git a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesAdd.scala b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesAdd.scala index c5703bc429556ce88ff84706bfc362aa91381c16..fc5cababd9c141208b2a8ec945895babaac0dc4a 100644 --- a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesAdd.scala +++ b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesAdd.scala @@ -20,8 +20,11 @@ class ManweSamplesAdd(val root: Configurable) extends Manwe { var poolSize: Option[Int] = _ def subCommand = { - required("samples") + required("add") + required(name) + - optional("-s", poolSize) + repeat("-g", group) + required("samples") + + required("add") + + required(name) + + optional("-s", poolSize) + + repeat("-g", group) } } diff --git a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesAnnotateVariations.scala b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesAnnotateVariations.scala index cecb02cf3e41f59f9f943beca01acfc549420620..04092cc14b1cd39fd8d8a006c77dfadfa9163627 100644 --- a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesAnnotateVariations.scala +++ b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesAnnotateVariations.scala @@ -17,8 +17,10 @@ class ManweSamplesAnnotateVariations(val root: Configurable) extends Manwe { var queries: List[String] = Nil def subCommand = { - required("samples") + required("annotate-variations") + - required(uri) + repeat("-q", queries) + required("samples") + + required("annotate-variations") + + required(uri) + + repeat("-q", queries) } } diff --git a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesImport.scala b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesImport.scala index 649b4faeb1c52c0379be9c7cf6e96c6d5ab515e4..e683f2c2a7d8e31481becf2d9ed4b6567efc744f 100644 --- a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesImport.scala +++ b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesImport.scala @@ -45,10 +45,16 @@ class ManweSamplesImport(val root: Configurable) extends Manwe { var waitToComplete: Boolean = false def subCommand = { - required("samples") + required("import") + required(name) + - repeat("-g", group) + repeat("--vcf", vcfs) + repeat("--bed", beds) + - optional("-s", poolSize) + conditional(alreadyUploaded, "-u") + - conditional(public, "-p") + conditional(preferLikelihood, "-l") + + required("samples") + + required("import") + + required(name) + + repeat("-g", group) + + repeat("--vcf", vcfs) + + repeat("--bed", beds) + + optional("-s", poolSize) + + conditional(alreadyUploaded, "-u") + + conditional(public, "-p") + + conditional(preferLikelihood, "-l") + conditional(noCoverage, "--no-coverage-profile") + conditional(waitToComplete, "--wait") } diff --git a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesImportBed.scala b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesImportBed.scala index fe07380b2ec78f23ea12c7374839339a83874139..9eb3f12d0974d95af940b0e34f3b930fd06ba0a1 100644 --- a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesImportBed.scala +++ b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesImportBed.scala @@ -27,9 +27,12 @@ class ManweSamplesImportBed(val root: Configurable) extends Manwe { var waitToComplete: Boolean = false def subCommand = { - required("samples") + required("import-bed") + - required(uri) + required(bed) + - conditional(alreadyUploaded, "-u") + conditional(waitToComplete, "--wait") + required("samples") + + required("import-bed") + + required(uri) + + required(bed) + + conditional(alreadyUploaded, "-u") + + conditional(waitToComplete, "--wait") } } diff --git a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesImportVcf.scala b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesImportVcf.scala index 582af63155765feb8af8fdf1f2929cbc8283b6bc..61f6a5f223b4af57d6066ae8f80cb0164cb8cd53 100644 --- a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesImportVcf.scala +++ b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesImportVcf.scala @@ -30,9 +30,11 @@ class ManweSamplesImportVcf(val root: Configurable) extends Manwe { var waitToComplete: Boolean = false def subCommand = { - required("samples") + required("import-vcf") + + required("samples") + + required("import-vcf") + required(uri) + required(vcf) + - conditional(alreadyUploaded, "-u") + conditional(preferLikelihoods, "-l") + + conditional(alreadyUploaded, "-u") + + conditional(preferLikelihoods, "-l") + conditional(waitToComplete, "--wait") } diff --git a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesList.scala b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesList.scala index d1befca4e391d2a5c2b4f64a9b1009ba8f8a4ab1..d79b85fe1048210830d532091e4d588dac9c96d4 100644 --- a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesList.scala +++ b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesList.scala @@ -19,8 +19,11 @@ class ManweSamplesList(val root: Configurable) extends Manwe { var onlyPublic: Boolean = false def subCommand = { - required("samples") + required("list") + optional("-u", user) + - repeat("-g", group) + conditional(onlyPublic, "-p") + required("samples") + + required("list") + + optional("-u", user) + + repeat("-g", group) + + conditional(onlyPublic, "-p") } } diff --git a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesShow.scala b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesShow.scala index 1cee462fc05e4d564f52ee4dd057dfbb8958fa12..6a73f84a44c44505509fe993a6fbca92da393b66 100644 --- a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesShow.scala +++ b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/manwe/ManweSamplesShow.scala @@ -14,7 +14,9 @@ class ManweSamplesShow(val root: Configurable) extends Manwe { var uri: Option[String] = _ def subCommand = { - required("samples") + required("show") + required(uri) + required("samples") + + required("show") + + required(uri) } } diff --git a/public/biopet-extensions/src/test/scala/nl/lumc/sasc/biopet/extensions/ManweTest.scala b/public/biopet-extensions/src/test/scala/nl/lumc/sasc/biopet/extensions/ManweTest.scala index 5046b0a2e4da8b037ed4fb5cd4fb5612f94cfdc7..6d1cf3d60c7f2b0d995b9b82b59a1391d37162db 100644 --- a/public/biopet-extensions/src/test/scala/nl/lumc/sasc/biopet/extensions/ManweTest.scala +++ b/public/biopet-extensions/src/test/scala/nl/lumc/sasc/biopet/extensions/ManweTest.scala @@ -334,14 +334,17 @@ class ManweTest extends TestNGSuite with Matchers { } val file: File = manwe.createManweConfig(None) - val contents = Source.fromFile(file).getLines().mkString("\n") + val contents = Source.fromFile(file).getLines().toList - contents should equal("""API_ROOT = 'http://127.0.0.1:5000' - |TOKEN = 'QWERTYUIOPASDFGHJKLZXCVBNM' - |VERIFY_CERTIFICATE = True - |COLLECTION_CACHE_SIZE = 25 - |DATA_BUFFER_SIZE = 200 - |TASK_POLL_WAIT = 5""".stripMargin) + val supposedContent = List("API_ROOT = 'http://127.0.0.1:5000'", + "TOKEN = 'QWERTYUIOPASDFGHJKLZXCVBNM'", + "VERIFY_CERTIFICATE = True", + "COLLECTION_CACHE_SIZE = 25", + "DATA_BUFFER_SIZE = 200", + "TASK_POLL_WAIT = 5" + ) + + supposedContent.sorted should equal(contents.sorted) val manwe2 = new ManweAnnotateBed(null) { override def globalConfig = new Config(Map( @@ -354,15 +357,16 @@ class ManweTest extends TestNGSuite with Matchers { } val file2: File = manwe2.createManweConfig(None) - val contents2 = Source.fromFile(file2).getLines().mkString("\n") - - contents2 should equal("""API_ROOT = 'http://127.0.0.1:5000' - |TOKEN = 'QWERTYUIOPASDFGHJKLZXCVBNM' - |VERIFY_CERTIFICATE = '/a/b/c/d.crt' - |COLLECTION_CACHE_SIZE = 25 - |DATA_BUFFER_SIZE = 200 - |TASK_POLL_WAIT = 5""".stripMargin) - + val contents2 = Source.fromFile(file2).getLines().toList + val supposedContent2 = List("API_ROOT = 'http://127.0.0.1:5000'", + "TOKEN = 'QWERTYUIOPASDFGHJKLZXCVBNM'", + "VERIFY_CERTIFICATE = '/a/b/c/d.crt'", + "COLLECTION_CACHE_SIZE = 25", + "DATA_BUFFER_SIZE = 200", + "TASK_POLL_WAIT = 5" + ) + + supposedContent2.sorted should equal(contents2.sorted) } } diff --git a/public/toucan/src/main/scala/nl/lumc/sasc/biopet/pipelines/toucan/Toucan.scala b/public/toucan/src/main/scala/nl/lumc/sasc/biopet/pipelines/toucan/Toucan.scala index 620192846e8b0352ebd4fbb905cf9a810774aa8c..6de6ef0d914868b177b4a30a29b1f721e0121065 100644 --- a/public/toucan/src/main/scala/nl/lumc/sasc/biopet/pipelines/toucan/Toucan.scala +++ b/public/toucan/src/main/scala/nl/lumc/sasc/biopet/pipelines/toucan/Toucan.scala @@ -186,7 +186,7 @@ class Toucan(val root: Configurable) extends QScript with BiopetQScript with Sum */ def varda(vcf: File, gVcf: File): File = { - val annotationQueries: List[String] = config("annotation_queries", default = Nil, submodule = "manwe") + val annotationQueries: List[String] = config("annotation_queries", default = List("GLOBAL *"), submodule = "manwe") //TODO: add groups!!! Need sample-specific group tags for this val annotate = new ManweAnnotateVcf(this)