diff --git a/protected/biopet-gatk-pipelines/src/test/scala/nl/lumc/sasc/biopet/pipelines/gatk/ShivaTest.scala b/protected/biopet-gatk-pipelines/src/test/scala/nl/lumc/sasc/biopet/pipelines/gatk/ShivaTest.scala index 66f0bfcc564503567782bcc1e35b0cc0899e6212..f03a5ca263af6b7f859a69d4627369375c14a6f6 100644 --- a/protected/biopet-gatk-pipelines/src/test/scala/nl/lumc/sasc/biopet/pipelines/gatk/ShivaTest.scala +++ b/protected/biopet-gatk-pipelines/src/test/scala/nl/lumc/sasc/biopet/pipelines/gatk/ShivaTest.scala @@ -115,7 +115,6 @@ object ShivaTest { "dir" -> "test", "vep_script" -> "test", "output_dir" -> outputDir, - "reference" -> (outputDir + File.separator + "ref.fa"), "reference_fasta" -> (outputDir + File.separator + "ref.fa"), "gatk_jar" -> "test", "samtools" -> Map("exe" -> "test"), diff --git a/protected/biopet-gatk-pipelines/src/test/scala/nl/lumc/sasc/biopet/pipelines/gatk/ShivaVariantcallingTest.scala b/protected/biopet-gatk-pipelines/src/test/scala/nl/lumc/sasc/biopet/pipelines/gatk/ShivaVariantcallingTest.scala index 8e8a1eddf9e5d6b5aab36c4c6fea7d35ea2da82e..1740b33ac108a4c20205973a2410aabbeb4dbd83 100644 --- a/protected/biopet-gatk-pipelines/src/test/scala/nl/lumc/sasc/biopet/pipelines/gatk/ShivaVariantcallingTest.scala +++ b/protected/biopet-gatk-pipelines/src/test/scala/nl/lumc/sasc/biopet/pipelines/gatk/ShivaVariantcallingTest.scala @@ -131,7 +131,6 @@ object ShivaVariantcallingTest { "cache" -> true, "dir" -> "test", "vep_script" -> "test", - "reference" -> (outputDir + File.separator + "ref.fa"), "reference_fasta" -> (outputDir + File.separator + "ref.fa"), "gatk_jar" -> "test", "samtools" -> Map("exe" -> "test"), diff --git a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/picard/CollectAlignmentSummaryMetrics.scala b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/picard/CollectAlignmentSummaryMetrics.scala index ff19efe11676d2ca66adc172790080f9a9823783..b2fb2097efefb107ab4f00c3cadde5e9285a2568 100644 --- a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/picard/CollectAlignmentSummaryMetrics.scala +++ b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/picard/CollectAlignmentSummaryMetrics.scala @@ -17,12 +17,13 @@ package nl.lumc.sasc.biopet.extensions.picard import java.io.File +import nl.lumc.sasc.biopet.core.Reference import nl.lumc.sasc.biopet.utils.config.Configurable import nl.lumc.sasc.biopet.core.summary.Summarizable import org.broadinstitute.gatk.utils.commandline.{ Argument, Input, Output } /** Extension for picard CollectAlignmentSummaryMetrics */ -class CollectAlignmentSummaryMetrics(val root: Configurable) extends Picard with Summarizable { +class CollectAlignmentSummaryMetrics(val root: Configurable) extends Picard with Summarizable with Reference { javaMainClass = new picard.analysis.CollectAlignmentSummaryMetrics().getClass.getName @Input(doc = "The input SAM or BAM files to analyze. Must be coordinate sorted.", required = true) @@ -41,7 +42,7 @@ class CollectAlignmentSummaryMetrics(val root: Configurable) extends Picard with var output: File = _ @Argument(doc = "Reference file", required = false) - var reference: File = config("reference") + var reference: File = _ @Argument(doc = "ASSUME_SORTED", required = false) var assumeSorted: Boolean = config("assumeSorted", default = true) @@ -52,6 +53,11 @@ class CollectAlignmentSummaryMetrics(val root: Configurable) extends Picard with @Argument(doc = "STOP_AFTER", required = false) var stopAfter: Option[Long] = config("stopAfter") + override def beforeGraph(): Unit = { + super.beforeGraph() + if (reference == null) reference = referenceFasta() + } + /** Returns command to execute */ override def cmdLine = super.cmdLine + required("INPUT=", input, spaceSeparated = false) + diff --git a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/picard/CollectInsertSizeMetrics.scala b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/picard/CollectInsertSizeMetrics.scala index 3c4693b5ead3929b9b1bf7eac95b69c3bed0c83c..88d2eab6091ef4dafa02b46395ac1c78188ee3c2 100644 --- a/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/picard/CollectInsertSizeMetrics.scala +++ b/public/biopet-extensions/src/main/scala/nl/lumc/sasc/biopet/extensions/picard/CollectInsertSizeMetrics.scala @@ -17,6 +17,7 @@ package nl.lumc.sasc.biopet.extensions.picard import java.io.File +import nl.lumc.sasc.biopet.core.Reference import nl.lumc.sasc.biopet.utils.config.Configurable import nl.lumc.sasc.biopet.core.summary.Summarizable import org.broadinstitute.gatk.utils.commandline.{ Argument, Input, Output } @@ -24,7 +25,7 @@ import org.broadinstitute.gatk.utils.commandline.{ Argument, Input, Output } import scala.collection.immutable.Nil /** Extension for picard CollectInsertSizeMetrics */ -class CollectInsertSizeMetrics(val root: Configurable) extends Picard with Summarizable { +class CollectInsertSizeMetrics(val root: Configurable) extends Picard with Summarizable with Reference { javaMainClass = new picard.analysis.CollectInsertSizeMetrics().getClass.getName @Input(doc = "The input SAM or BAM files to analyze. Must be coordinate sorted.", required = true) @@ -37,7 +38,7 @@ class CollectInsertSizeMetrics(val root: Configurable) extends Picard with Summa protected var outputHistogram: File = null @Argument(doc = "Reference file", required = false) - var reference: File = config("reference") + var reference: File = _ @Argument(doc = "DEVIATIONS", required = false) var deviations: Option[Double] = config("deviations") @@ -59,6 +60,7 @@ class CollectInsertSizeMetrics(val root: Configurable) extends Picard with Summa override def beforeGraph() { outputHistogram = new File(output + ".pdf") + if (reference == null) reference = referenceFasta() } /** Returns command to execute */ diff --git a/public/carp/src/test/scala/nl/lumc/sasc/biopet/pipelines/carp/CarpTest.scala b/public/carp/src/test/scala/nl/lumc/sasc/biopet/pipelines/carp/CarpTest.scala index 1f5efdc2fb950dfca40e7a870bba7da984db0c41..da5d79939fd3729fb8838ab26bbe1a2d2d2bfb00 100644 --- a/public/carp/src/test/scala/nl/lumc/sasc/biopet/pipelines/carp/CarpTest.scala +++ b/public/carp/src/test/scala/nl/lumc/sasc/biopet/pipelines/carp/CarpTest.scala @@ -116,7 +116,6 @@ object CarpTest { copyFile("ref.fa.fai") val executables = Map( - "reference" -> (outputDir + File.separator + "ref.fa"), "reference_fasta" -> (outputDir + File.separator + "ref.fa"), "fastqc" -> Map("exe" -> "test"), "seqtk" -> Map("exe" -> "test"), diff --git a/public/gentrap/src/test/scala/nl/lumc/sasc/biopet/pipelines/gentrap/GentrapTest.scala b/public/gentrap/src/test/scala/nl/lumc/sasc/biopet/pipelines/gentrap/GentrapTest.scala index 37deb389eb94f4f04cd22812b600dab8f9adddbe..575b20b3e9fe58ab5c1caca50aeb1308b6066bdf 100644 --- a/public/gentrap/src/test/scala/nl/lumc/sasc/biopet/pipelines/gentrap/GentrapTest.scala +++ b/public/gentrap/src/test/scala/nl/lumc/sasc/biopet/pipelines/gentrap/GentrapTest.scala @@ -196,7 +196,6 @@ object GentrapTest { copyFile("ref.fa.fai") val executables = Map( - "reference" -> (outputDir + File.separator + "ref.fa"), "reference_fasta" -> (outputDir + File.separator + "ref.fa"), "refFlat" -> "test", "annotation_gtf" -> "test", diff --git a/public/shiva/src/test/scala/nl/lumc/sasc/biopet/pipelines/shiva/ShivaTest.scala b/public/shiva/src/test/scala/nl/lumc/sasc/biopet/pipelines/shiva/ShivaTest.scala index 53286ecde3d44e4e6ad91fe142a5740b4909ceca..31777668d11df48a3ad99a663d60aad6e7666966 100644 --- a/public/shiva/src/test/scala/nl/lumc/sasc/biopet/pipelines/shiva/ShivaTest.scala +++ b/public/shiva/src/test/scala/nl/lumc/sasc/biopet/pipelines/shiva/ShivaTest.scala @@ -109,7 +109,6 @@ object ShivaTest { "cache" -> true, "dir" -> "test", "vep_script" -> "test", - "reference" -> (outputDir + File.separator + "ref.fa"), "reference_fasta" -> (outputDir + File.separator + "ref.fa"), "gatk_jar" -> "test", "samtools" -> Map("exe" -> "test"), diff --git a/public/shiva/src/test/scala/nl/lumc/sasc/biopet/pipelines/shiva/ShivaVariantcallingTest.scala b/public/shiva/src/test/scala/nl/lumc/sasc/biopet/pipelines/shiva/ShivaVariantcallingTest.scala index caeea0326d46e4b7673dae4a3d07f27c3cbd5f4e..dae0b974329950501369990e3a53bae077f8f071 100644 --- a/public/shiva/src/test/scala/nl/lumc/sasc/biopet/pipelines/shiva/ShivaVariantcallingTest.scala +++ b/public/shiva/src/test/scala/nl/lumc/sasc/biopet/pipelines/shiva/ShivaVariantcallingTest.scala @@ -112,7 +112,6 @@ object ShivaVariantcallingTest { "cache" -> true, "dir" -> "test", "vep_script" -> "test", - "reference" -> (outputDir + File.separator + "ref.fa"), "reference_fasta" -> (outputDir + File.separator + "ref.fa"), "gatk_jar" -> "test", "samtools" -> Map("exe" -> "test"),