diff --git a/protected/biopet-gatk-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/gatk/GatkGeneral.scala b/protected/biopet-gatk-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/gatk/GatkGeneral.scala
index 147398ac798c077da0722b2078f7ea21c666b7d1..3d1b5585028f69c01934f0ddfc0d54c6a9c25169 100644
--- a/protected/biopet-gatk-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/gatk/GatkGeneral.scala
+++ b/protected/biopet-gatk-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/gatk/GatkGeneral.scala
@@ -13,13 +13,13 @@ trait GatkGeneral extends CommandLineGATK with BiopetJavaCommandLineFunction {
override def subPath = "gatk" :: super.subPath
- jarFile = config("gatk_jar", required = true)
+ jarFile = config("gatk_jar")
override val defaultVmem = "7G"
if (config.contains("intervals")) intervals = config("intervals").asFileList
if (config.contains("exclude_intervals")) excludeIntervals = config("exclude_intervals").asFileList
- reference_sequence = config("reference", required = true)
+ reference_sequence = config("reference")
if (config.contains("gatk_key")) gatk_key = config("gatk_key")
if (config.contains("pedigree")) pedigree = config("pedigree").asFileList
}
diff --git a/protected/biopet-gatk-pipelines/src/main/scala/nl/lumc/sasc/biopet/pipelines/gatk/GatkPipeline.scala b/protected/biopet-gatk-pipelines/src/main/scala/nl/lumc/sasc/biopet/pipelines/gatk/GatkPipeline.scala
index c76139133f23cf434884ad69ccc9dafc69b2cac9..be4566211351a3d077877d1a99cf8be6ba0f58cd 100644
--- a/protected/biopet-gatk-pipelines/src/main/scala/nl/lumc/sasc/biopet/pipelines/gatk/GatkPipeline.scala
+++ b/protected/biopet-gatk-pipelines/src/main/scala/nl/lumc/sasc/biopet/pipelines/gatk/GatkPipeline.scala
@@ -35,7 +35,7 @@ class GatkPipeline(val root: Configurable) extends QScript with MultiSampleQScri
var jointGenotyping: Boolean = config("joint_genotyping", default = false)
var singleSampleCalling = config("single_sample_calling", default = true)
- var reference: File = config("reference", required = true)
+ var reference: File = config("reference")
var useAllelesOption: Boolean = config("use_alleles_option", default = false)
val externalGvcfs = config("external_gvcfs_files", default = Nil).asFileList
diff --git a/protected/biopet-gatk-pipelines/src/main/scala/nl/lumc/sasc/biopet/pipelines/gatk/GatkVariantcalling.scala b/protected/biopet-gatk-pipelines/src/main/scala/nl/lumc/sasc/biopet/pipelines/gatk/GatkVariantcalling.scala
index 8bac4aaf68c33a245da877d460bc26abb9ebe564..47715e26adb11f0ed73520d2769ca55120a58d93 100644
--- a/protected/biopet-gatk-pipelines/src/main/scala/nl/lumc/sasc/biopet/pipelines/gatk/GatkVariantcalling.scala
+++ b/protected/biopet-gatk-pipelines/src/main/scala/nl/lumc/sasc/biopet/pipelines/gatk/GatkVariantcalling.scala
@@ -30,7 +30,7 @@ class GatkVariantcalling(val root: Configurable) extends QScript with BiopetQScr
var rawVcfInput: File = _
@Argument(doc = "Reference", shortName = "R", required = false)
- var reference: File = config("reference", required = true)
+ var reference: File = config("reference")
@Argument(doc = "OutputName", required = false)
var outputName: String = _
diff --git a/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/core/BiopetQScript.scala b/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/core/BiopetQScript.scala
index 1a3f565880dc3a2bd5450efd86cfa188e647eef4..1e94c21be246afada48771cd0d351452a62f2d41 100644
--- a/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/core/BiopetQScript.scala
+++ b/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/core/BiopetQScript.scala
@@ -51,7 +51,7 @@ trait BiopetQScript extends Configurable with GatkLogging {
var functions: Seq[QFunction]
final def script() {
- outputDir = config("output_dir", required = true)
+ outputDir = config("output_dir")
if (!outputDir.endsWith("/")) outputDir += "/"
init
biopetScript
diff --git a/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/Bowtie.scala b/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/Bowtie.scala
index e192a845be6db96d2a754d2d2ff6a5e393d136ca..9ac74448e880d7be01063a3f12bdd613141b34a4 100644
--- a/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/Bowtie.scala
+++ b/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/Bowtie.scala
@@ -29,7 +29,7 @@ class Bowtie(val root: Configurable) extends BiopetCommandLineFunction {
var R2: File = _
@Input(doc = "The reference file for the bam files.", shortName = "R")
- var reference: File = config("reference", required = true)
+ var reference: File = config("reference")
@Output(doc = "Output file SAM", shortName = "output")
var output: File = _
diff --git a/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/Stampy.scala b/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/Stampy.scala
index a7bfeab78447e62eee1634c73ca09c880679327e..1db8f6d67c8d892e6188aa750efb40b814fbbf1e 100644
--- a/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/Stampy.scala
+++ b/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/Stampy.scala
@@ -29,13 +29,13 @@ class Stampy(val root: Configurable) extends BiopetCommandLineFunction {
var R2: File = _
@Input(doc = "The reference file for the bam files.", shortName = "ref")
- var reference: File = config("reference", required = true)
+ var reference: File = config("reference")
@Input(doc = "The genome prefix.")
- var genome: File = config("genome", required = true)
+ var genome: File = config("genome")
@Input(doc = "The hash prefix")
- var hash: File = config("hash", required = true)
+ var hash: File = config("hash")
@Output(doc = "Output file SAM", shortName = "output")
var output: File = _
diff --git a/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/TopHat.scala b/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/TopHat.scala
index e0b8bd68bda46a6b3dc6915a54482b093cfd43d6..98ee6de521d499710e186bb2c196d9d471c6aa5d 100644
--- a/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/TopHat.scala
+++ b/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/TopHat.scala
@@ -29,7 +29,7 @@ class TopHat(val root: Configurable) extends BiopetCommandLineFunction {
var R2: File = _
@Input(doc = "Bowtie index", shortName = "bti")
- var bowtie_index: File = config("bowtie_index", required = true)
+ var bowtie_index: File = config("bowtie_index")
@Argument(doc = "Output Directory")
var outputDir: String = _
diff --git a/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/bwa/BwaAln.scala b/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/bwa/BwaAln.scala
index bfd0a5846e6531b47f285453a2d848b1b7f1bafe..3298702fcfd5cd98e1eaf0bae8f6b059d353d2c9 100644
--- a/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/bwa/BwaAln.scala
+++ b/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/bwa/BwaAln.scala
@@ -13,7 +13,7 @@ class BwaAln(val root: Configurable) extends Bwa {
var fastq: File = _
@Input(doc = "The reference file for the bam files.", required = true)
- var reference: File = config("reference", required = true)
+ var reference: File = config("reference")
@Output(doc = "Output file SAM", required = false)
var output: File = _
diff --git a/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/bwa/BwaMem.scala b/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/bwa/BwaMem.scala
index fc790b183b5bae2922a1b5f89ec66fcf4f5b85b2..74cdad0cea290e5a8cdcc2b360fa3fa32444791c 100644
--- a/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/bwa/BwaMem.scala
+++ b/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/bwa/BwaMem.scala
@@ -29,7 +29,7 @@ class BwaMem(val root: Configurable) extends Bwa {
var R2: File = _
@Input(doc = "The reference file for the bam files.", shortName = "R")
- var reference: File = config("reference", required = true)
+ var reference: File = config("reference")
@Output(doc = "Output file SAM", shortName = "output")
var output: File = _
diff --git a/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/bwa/BwaSampe.scala b/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/bwa/BwaSampe.scala
index b857eea014ac52acb9608debb94db9cd75cef929..255811561e732b229b1417c95d866fad4694c0c9 100644
--- a/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/bwa/BwaSampe.scala
+++ b/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/bwa/BwaSampe.scala
@@ -26,7 +26,7 @@ class BwaSampe(val root: Configurable) extends Bwa {
var saiR2: File = _
@Input(doc = "The reference file for the bam files.", required = true)
- var reference: File = config("reference", required = true)
+ var reference: File = config("reference")
@Output(doc = "Output file SAM", required = false)
var output: File = _
diff --git a/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/bwa/BwaSamse.scala b/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/bwa/BwaSamse.scala
index 51f9a0f30eaf73a897fefaa2b6cf69d4e3386b62..8bbf918474e21d2e2837ce01d5bc4490e13bd087 100644
--- a/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/bwa/BwaSamse.scala
+++ b/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/bwa/BwaSamse.scala
@@ -16,7 +16,7 @@ class BwaSamse(val root: Configurable) extends Bwa {
var sai: File = _
@Input(doc = "The reference file for the bam files.", required = true)
- var reference: File = config("reference", required = true)
+ var reference: File = config("reference")
@Output(doc = "Output file SAM", required = false)
var output: File = _
diff --git a/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/picard/CollectGcBiasMetrics.scala b/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/picard/CollectGcBiasMetrics.scala
index 7ffcb50d8fec02138c07e4fc95d7608e8bfba72e..9cab9bfc90298f243f6cfdc8e051e7d451a4cf90 100644
--- a/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/picard/CollectGcBiasMetrics.scala
+++ b/public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/picard/CollectGcBiasMetrics.scala
@@ -35,7 +35,7 @@ class CollectGcBiasMetrics(val root: Configurable) extends Picard {
var outputSummary: File = _
@Argument(doc = "Reference file", required = false)
- var reference: File = config("reference", required = true)
+ var reference: File = config("reference")
@Argument(doc = "Window size", required = false)
var windowSize: Option[Int] = config("windowsize")
diff --git a/public/biopet-framework/src/test/scala/nl/lumc/sasc/biopet/core/config/ConfigTest.scala b/public/biopet-framework/src/test/scala/nl/lumc/sasc/biopet/core/config/ConfigTest.scala
index 92d0b9f63f60ad1a6c9489ba1d71fd3f25ab6828..b7353b2ad1d4a91cd05340edbd7a61a51e64e266 100644
--- a/public/biopet-framework/src/test/scala/nl/lumc/sasc/biopet/core/config/ConfigTest.scala
+++ b/public/biopet-framework/src/test/scala/nl/lumc/sasc/biopet/core/config/ConfigTest.scala
@@ -30,9 +30,6 @@ class ConfigTest extends TestNGSuite with Matchers with ConfigUtils.ImplicitConv
@Test def testApply: Unit = {
ConfigTest.config("m1", Nil, "k1").asString shouldBe "v2"
ConfigTest.config("m1", Nil, "notexist", default = "default").asString shouldBe "default"
- intercept[IllegalStateException] {
- ConfigTest.config("m1", Nil, "notexist")
- }
}
@Test def testMergeConfigs: Unit = {
diff --git a/public/biopet-framework/src/test/scala/nl/lumc/sasc/biopet/core/config/ConfigurableTest.scala b/public/biopet-framework/src/test/scala/nl/lumc/sasc/biopet/core/config/ConfigurableTest.scala
index 1b493f2e14df822b75534fed3722f88917882fe1..55f672cc1dad99011d0fdd38bec9abd4f49c1e82 100644
--- a/public/biopet-framework/src/test/scala/nl/lumc/sasc/biopet/core/config/ConfigurableTest.scala
+++ b/public/biopet-framework/src/test/scala/nl/lumc/sasc/biopet/core/config/ConfigurableTest.scala
@@ -27,10 +27,6 @@ class ConfigurableTest extends TestNGSuite with Matchers {
classC.classB.get("k1").asString shouldBe "c1"
classC.classB.classA.get("k1").asString shouldBe "c1"
- classC.get("notexist") shouldBe null
- intercept[IllegalStateException] {
- classC.get("notexist", required = true)
- }
classC.get("notexist", default = "default").asString shouldBe "default"
classC.get("k1", freeVar = false).asString shouldBe "c1"
@@ -47,11 +43,10 @@ abstract class Cfg extends Configurable {
def get(key: String,
default: String = null,
submodule: String = null,
- required: Boolean = false,
freeVar: Boolean = true,
sample: String = null,
library: String = null) = {
- config(key, default, submodule, required, freeVar = freeVar, sample = sample, library = library)
+ config(key, default, submodule, freeVar = freeVar, sample = sample, library = library)
}
}
diff --git a/public/sage/src/main/scala/nl/lumc/sasc/biopet/pipelines/sage/Sage.scala b/public/sage/src/main/scala/nl/lumc/sasc/biopet/pipelines/sage/Sage.scala
index ed55ca382d13346b3cbb7bb4862941ef4da591f5..77dba4b1469eba2d75bedafc9f53be7c23acb557 100644
--- a/public/sage/src/main/scala/nl/lumc/sasc/biopet/pipelines/sage/Sage.scala
+++ b/public/sage/src/main/scala/nl/lumc/sasc/biopet/pipelines/sage/Sage.scala
@@ -60,7 +60,7 @@ class Sage(val root: Configurable) extends QScript with MultiSampleQScript {
class Sample(sampleId: String) extends AbstractSample(sampleId) {
def makeLibrary(id: String) = new Library(id)
class Library(libId: String) extends AbstractLibrary(libId) {
- val inputFastq: File = config("R1", required = true)
+ val inputFastq: File = config("R1")
val prefixFastq: File = createFile(".prefix.fastq")
val flexiprep = new Flexiprep(qscript)
diff --git a/public/yamsvp/src/main/scala/nl/lumc/sasc/biopet/pipelines/yamsvp/Yamsvp.scala b/public/yamsvp/src/main/scala/nl/lumc/sasc/biopet/pipelines/yamsvp/Yamsvp.scala
index 206c92a69453105fb2034d6cd72cce00f5ab75cd..6e7257c7d94967eac350fc212e7c7b37ebf5dc58 100644
--- a/public/yamsvp/src/main/scala/nl/lumc/sasc/biopet/pipelines/yamsvp/Yamsvp.scala
+++ b/public/yamsvp/src/main/scala/nl/lumc/sasc/biopet/pipelines/yamsvp/Yamsvp.scala
@@ -35,7 +35,7 @@ import org.broadinstitute.gatk.queue.engine.JobRunInfo
class Yamsvp(val root: Configurable) extends QScript with BiopetQScript { //with MultiSampleQScript {
def this() = this(null)
- var reference: File = config("reference", required = true)
+ var reference: File = config("reference")
var finalBamFiles: List[File] = Nil
/*
class LibraryOutput extends AbstractLibraryOutput {