Merge remote-tracking branch 'origin/develop' into feature-external_examples

README.md

@@ -46,7 +46,7 @@ If the dry run proceeds without problems, you can then do the real run by using
 $ biopet pipeline <pipeline_name> -config <path/to/config.json> -qsub -jobParaEnv BWA -retry 2 -run
 ~~~
 
-It is usually a good idea to do the real run using `screen` or `nohup` to prevent the job from terminating when you log out of SHARK. In practice, using `biopet` as it is is also fine. What you need to keep in mind, is that each pipeline has their own expected config layout. You can check out more about the general structure of our config files [here](general/config.md). For the specific structure that each pipeline accepts, please consult the respective pipeline page.
+It is usually a good idea to do the real run using `screen` or `nohup` to prevent the job from terminating when you log out of SHARK. In practice, using `biopet` as it is is also fine. What you need to keep in mind, is that each pipeline has their own expected config layout. You can check out more about the general structure of our config files [here](docs/config.md). For the specific structure that each pipeline accepts, please consult the respective pipeline page.
 
 ### Running Biopet in your own computer
 
@@ -55,25 +55,25 @@ At the moment, we do not provide links to download the Biopet package. If you ar
 
 ## Contributing to Biopet
 
-Biopet is based on the Queue framework developed by the Broad Institute as part of their Genome Analysis Toolkit (GATK) framework. The current Biopet release is based on the GATK 3.3 release.
+Biopet is based on the Queue framework developed by the Broad Institute as part of their Genome Analysis Toolkit (GATK) framework. The current Biopet release is based on the GATK 3.4 release.
 
-We welcome any kind of contribution, be it merge requests on the code base, documentation updates, or any kinds of other fixes! The main language we use is Scala, though the repository also contains a small bit of Python and R. Our main code repository is located at [https://git.lumc.nl/biopet/biopet](https://git.lumc.nl/biopet/biopet/issues), along with our issue tracker.
+We welcome any kind of contribution, be it merge requests on the code base, documentation updates, or any kinds of other fixes! The main language we use is Scala, though the repository also contains a small bit of Python and R. Our main code repository is located at [https://github.com/biopet/biopet](https://github.com/biopet/biopet/issues), along with our issue tracker.
 
 ## Local development setup
 
-To develop Biopet, Java 7, Maven 3.2.2, and GATK Queue 3.3 is required. Please consult the Java homepage and Maven homepage for the respective installation instruction. After you have both Java and Maven installed, you would then need to install GATK Queue. However, as the GATK Queue package is not yet available as an artifact in Maven Central, you will need to download, compile, and install GATK Queue first.
+To develop Biopet, Java 7, Maven 3.2.2, and GATK Queue 3.4 is required. Please consult the Java homepage and Maven homepage for the respective installation instruction. After you have both Java and Maven installed, you would then need to install GATK Queue. However, as the GATK Queue package is not yet available as an artifact in Maven Central, you will need to download, compile, and install GATK Queue first.
 
 ~~~
 $ git clone https://github.com/broadgsa/gatk
 $ cd gatk
-$ git checkout 3.3                              # the current release is based on GATK 3.3
+$ git checkout 3.4                              # the current release is based on GATK 3.4
 $ mvn -U clean install
 ~~~
 
 This will install all the required dependencies to your local maven repository. After this is done, you can clone our repository and test if everything builds fine:
 
 ~~~
-$ git clone git@git.lumc.nl:biopet/biopet.git
+$ git clone https://github.com/biopet/biopet.git
 $ cd biopet
 $ mvn -U clean install
 ~~~
@@ -83,8 +83,8 @@ If everything builds fine, you're good to go! Otherwise, don't hesitate to conta
 
 ## About
 
-Go to the [about page](about)
+Go to the [about page](docs/about.md)
 
 ## License
 
-See: [License](license.md)
+See: [License](docs/license.md)

docs/index.md

-../README.md
\ No newline at end of file

docs/index.md

+# Welcome to Biopet
+
+
+## Introduction
+
+Biopet (Bio Pipeline Execution Toolkit) is the main pipeline development framework of the LUMC Sequencing Analysis Support Core team. It contains our main pipelines and some of the command line tools we develop in-house. It is meant to be used in the main [SHARK](https://humgenprojects.lumc.nl/trac/shark) computing cluster. While usage outside of SHARK is technically possible, some adjustments may need to be made in order to do so.
+
+
+## Quick Start
+
+### Running Biopet in the SHARK cluster
+
+Biopet is available as a JAR package in SHARK. The easiest way to start using it is to activate the `biopet` environment module, which sets useful aliases and environment variables:
+
+~~~
+$ module load biopet/v0.4.0
+~~~
+
+With each Biopet release, an accompanying environment module is also released. The latest release is version 0.4.0, thus `biopet/v0.4.0` is the module you would want to load.
+
+After loading the module, you can access the biopet package by simply typing `biopet`:
+
+~~~
+$ biopet
+~~~
+
+This will show you a list of tools and pipelines that you can use straight away. You can also execute `biopet pipeline` to show only available pipelines or `biopet tool` to show only the tools. What you should be aware of, is that this is actually a shell function that calls `java` on the system-wide available Biopet JAR file.
+
+~~~
+$ java -jar <path/to/current/biopet/release.jar>
+~~~
+
+The actual path will vary from version to version, which is controlled by which module you loaded.
+
+Almost all of the pipelines have a common usage pattern with a similar set of flags, for example:
+
+~~~
+$ biopet pipeline <pipeline_name> -config <path/to/config.json> -qsub -jobParaEnv BWA -retry 2
+~~~
+
+The command above will do a *dry* run of a pipeline using a config file as if the command would be submitted to the SHARK cluster (the `-qsub` flag) to the `BWA` parallel environment (the `-jobParaEnv BWA` flag). We also set the maximum retry of failing jobs to two times (via the `-retry 2` flag). Doing a good run is a good idea to ensure that your real run proceeds smoothly. It may not catch all the errors, but if the dry run fails you can be sure that the real run will never succeed.
+
+If the dry run proceeds without problems, you can then do the real run by using the `-run` flag:
+
+~~~
+$ biopet pipeline <pipeline_name> -config <path/to/config.json> -qsub -jobParaEnv BWA -retry 2 -run
+~~~
+
+It is usually a good idea to do the real run using `screen` or `nohup` to prevent the job from terminating when you log out of SHARK. In practice, using `biopet` as it is is also fine. What you need to keep in mind, is that each pipeline has their own expected config layout. You can check out more about the general structure of our config files [here](general/config.md). For the specific structure that each pipeline accepts, please consult the respective pipeline page.
+
+### Running Biopet in your own computer
+
+At the moment, we do not provide links to download the Biopet package. If you are interested in trying out Biopet locally, please contact us as [sasc@lumc.nl](mailto:sasc@lumc.nl).
+
+
+## Contributing to Biopet
+
+Biopet is based on the Queue framework developed by the Broad Institute as part of their Genome Analysis Toolkit (GATK) framework. The current Biopet release is based on the GATK 3.4 release.
+
+We welcome any kind of contribution, be it merge requests on the code base, documentation updates, or any kinds of other fixes! The main language we use is Scala, though the repository also contains a small bit of Python and R. Our main code repository is located at [https://github.com/biopet/biopet](https://github.com/biopet/biopet/issues), along with our issue tracker.
+
+## Local development setup
+
+To develop Biopet, Java 7, Maven 3.2.2, and GATK Queue 3.4 is required. Please consult the Java homepage and Maven homepage for the respective installation instruction. After you have both Java and Maven installed, you would then need to install GATK Queue. However, as the GATK Queue package is not yet available as an artifact in Maven Central, you will need to download, compile, and install GATK Queue first.
+
+~~~
+$ git clone https://github.com/broadgsa/gatk
+$ cd gatk
+$ git checkout 3.4                              # the current release is based on GATK 3.4
+$ mvn -U clean install
+~~~
+
+This will install all the required dependencies to your local maven repository. After this is done, you can clone our repository and test if everything builds fine:
+
+~~~
+$ git clone https://github.com/biopet/biopet.git
+$ cd biopet
+$ mvn -U clean install
+~~~
+
+If everything builds fine, you're good to go! Otherwise, don't hesitate to contact us or file an issue at our issue tracker.
+
+
+## About
+
+Go to the [about page](about.md)
+
+## License
+
+See: [License](license.md)

mkdocs.yml

@@ -39,4 +39,4 @@ pages:
 - License: 'license.md'
 #- ['developing/Setup.md', 'Developing', 'Setting up your local development environment']
 #theme: readthedocs
-repo_url: https://git.lumc.nl/biopet/biopet
+repo_url: https://github.com/biopet/biopet

pom.xml

@@ -9,7 +9,7 @@
     <parent>
         <groupId>nl.lumc.sasc</groupId>
         <artifactId>Biopet</artifactId>
-        <version>0.5.0-DEV</version>
+        <version>0.5.0-SNAPSHOT</version>
         <relativePath>public</relativePath>
     </parent>
     

protected/biopet-gatk-extensions/pom.xml

@@ -15,7 +15,7 @@
     <parent>
         <groupId>nl.lumc.sasc</groupId>
         <artifactId>BiopetGatk</artifactId>
-        <version>0.5.0-DEV</version>
+        <version>0.5.0-SNAPSHOT</version>
         <relativePath>../</relativePath>
     </parent>
     

protected/biopet-gatk-pipelines/pom.xml

@@ -15,7 +15,7 @@
     <parent>
         <groupId>nl.lumc.sasc</groupId>
         <artifactId>BiopetGatk</artifactId>
-        <version>0.5.0-DEV</version>
+        <version>0.5.0-SNAPSHOT</version>
         <relativePath>../</relativePath>
     </parent>
     

protected/biopet-protected-package/pom.xml

@@ -15,7 +15,7 @@
     <parent>
         <groupId>nl.lumc.sasc</groupId>
         <artifactId>BiopetGatk</artifactId>
-        <version>0.5.0-DEV</version>
+        <version>0.5.0-SNAPSHOT</version>
         <relativePath>../</relativePath>
     </parent>
     

protected/pom.xml

@@ -11,7 +11,7 @@
     <parent>
         <groupId>nl.lumc.sasc</groupId>
         <artifactId>BiopetRoot</artifactId>
-        <version>0.5.0-DEV</version>
+        <version>0.5.0-SNAPSHOT</version>
         <relativePath>../</relativePath>
     </parent>
     <artifactId>BiopetGatk</artifactId>

public/bam2wig/pom.xml

@@ -27,7 +27,7 @@
     <parent>
         <groupId>nl.lumc.sasc</groupId>
         <artifactId>Biopet</artifactId>
-        <version>0.5.0-DEV</version>
+        <version>0.5.0-SNAPSHOT</version>
         <relativePath>../</relativePath>
     </parent>
 

public/bammetrics/pom.xml

@@ -25,7 +25,7 @@
     <parent>
         <groupId>nl.lumc.sasc</groupId>
         <artifactId>Biopet</artifactId>
-        <version>0.5.0-DEV</version>
+        <version>0.5.0-SNAPSHOT</version>
         <relativePath>../</relativePath>
     </parent>
     

public/basty/pom.xml

@@ -32,7 +32,7 @@
     <parent>
         <groupId>nl.lumc.sasc</groupId>
         <artifactId>Biopet</artifactId>
-        <version>0.5.0-DEV</version>
+        <version>0.5.0-SNAPSHOT</version>
         <relativePath>../</relativePath>
     </parent>
     

public/biopet-framework/pom.xml

@@ -25,7 +25,7 @@
     <parent>
         <groupId>nl.lumc.sasc</groupId>
         <artifactId>Biopet</artifactId>
-        <version>0.5.0-DEV</version>
+        <version>0.5.0-SNAPSHOT</version>
         <relativePath>../</relativePath>
     </parent>
     
@@ -61,7 +61,7 @@
         <dependency>
             <groupId>org.scala-lang</groupId>
             <artifactId>scala-library</artifactId>
-            <version>2.10.2</version>
+            <version>2.10.4</version>
         </dependency>
         <dependency>
             <groupId>org.broadinstitute.gatk</groupId>

public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/Ln.scala

@@ -51,20 +51,14 @@ class Ln(val root: Configurable) extends InProcessFunction with Configurable {
   lazy val cmd: String = {
     lazy val inCanonical: String = {
       // need to remove "/~" to correctly expand path with tilde
-      input.getCanonicalPath.replace("/~", "")
+      input.getAbsolutePath.replace("/~", "")
     }
 
-    lazy val outCanonical: String = {
-      output.getCanonicalPath.replace("/~", "")
-    }
+    lazy val outCanonical: String = output.getAbsolutePath.replace("/~", "")
 
-    lazy val inToks: Array[String] = {
-      inCanonical.split(File.separator)
-    }
+    lazy val inToks: Array[String] = inCanonical.split(File.separator)
 
-    lazy val outToks: Array[String] = {
-      outCanonical.split(File.separator)
-    }
+    lazy val outToks: Array[String] = outCanonical.split(File.separator)
 
     lazy val commonPrefixLength: Int = {
       val maxLength = scala.math.min(inToks.length, outToks.length)

public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/extensions/sambamba/SambambaMarkdup.scala

@@ -60,7 +60,7 @@ object SambambaMarkdup {
   }
 
   def apply(root: Configurable, input: File): SambambaMarkdup = {
-    apply(root, input, new File(swapExtension(input.getCanonicalPath)))
+    apply(root, input, new File(swapExtension(input.getAbsolutePath)))
   }
 
   private def swapExtension(inputFile: String) = inputFile.stripSuffix(".bam") + ".dedup.bam"

public/biopet-framework/src/main/scala/nl/lumc/sasc/biopet/tools/VepNormalizer.scala

@@ -48,7 +48,7 @@ class VepNormalizer(val root: Configurable) extends ToolCommandFuntion {
   var outputVcf: File = null
 
   var mode: String = config("mode", default = "standard")
-  var doNotRemove: Boolean = config("donotremove", default = false)
+  var doNotRemove: Boolean = config("do_not_remove", default = false)
 
   override def defaultCoreMemory = 1.0
 

public/biopet-public-package/pom.xml

@@ -25,7 +25,7 @@
     <parent>
         <groupId>nl.lumc.sasc</groupId>
         <artifactId>Biopet</artifactId>
-        <version>0.5.0-DEV</version>
+        <version>0.5.0-SNAPSHOT</version>
         <relativePath>../</relativePath>
     </parent>
     

public/carp/pom.xml

@@ -25,7 +25,7 @@
     <parent>
         <groupId>nl.lumc.sasc</groupId>
         <artifactId>Biopet</artifactId>
-        <version>0.5.0-DEV</version>
+        <version>0.5.0-SNAPSHOT</version>
         <relativePath>../</relativePath>
     </parent>
     

public/flexiprep/pom.xml

@@ -25,7 +25,7 @@
     <parent>
         <groupId>nl.lumc.sasc</groupId>
         <artifactId>Biopet</artifactId>
-        <version>0.5.0-DEV</version>
+        <version>0.5.0-SNAPSHOT</version>
         <relativePath>../</relativePath>
     </parent>
     

public/flexiprep/src/main/scala/nl/lumc/sasc/biopet/pipelines/flexiprep/Flexiprep.scala

@@ -37,13 +37,19 @@ class Flexiprep(val root: Configurable) extends QScript with SummaryQScript with
   /** Skip Clip fastq files */
   var skipClip: Boolean = config("skip_clip", default = false)
 
+  /** Make a final fastq files, by default only when flexiprep is the main pipeline */
+  var keepQcFastqFiles: Boolean = config("keepQcFastqFiles", default = root == null)
+
   /** Location of summary file */
   def summaryFile = new File(outputDir, sampleId.getOrElse("x") + "-" + libId.getOrElse("x") + ".qc.summary.json")
 
   /** Returns files to store in summary */
   def summaryFiles: Map[String, File] = {
-    Map("input_R1" -> input_R1, "output_R1" -> outputFiles("output_R1_gzip")) ++
-      (if (paired) Map("input_R2" -> input_R2.get, "output_R2" -> outputFiles("output_R2_gzip")) else Map())
+    if (!skipTrim || !skipClip)
+      Map("input_R1" -> input_R1, "output_R1" -> outputFiles("output_R1_gzip")) ++
+        (if (paired) Map("input_R2" -> input_R2.get, "output_R2" -> outputFiles("output_R2_gzip")) else Map())
+    else Map("input_R1" -> input_R1) ++
+      (if (paired) Map("input_R2" -> input_R2.get) else Map())
   }
 
   /** returns settings to store in summary */
@@ -264,13 +270,13 @@ class Flexiprep(val root: Configurable) extends QScript with SummaryQScript with
     val R1 = new File(outputDir, R1_name + ".qc" + R1_ext + ".gz")
     val R2 = new File(outputDir, R2_name + ".qc" + R2_ext + ".gz")
 
-    add(Gzip(this, fastq_R1, R1))
-    if (paired) add(Gzip(this, fastq_R2, R2))
+    if (!skipTrim || !skipClip) {
+      add(Gzip(this, fastq_R1, R1), !keepQcFastqFiles)
+      if (paired) add(Gzip(this, fastq_R2, R2), !keepQcFastqFiles)
 
-    outputFiles += ("output_R1_gzip" -> R1)
-    if (paired) outputFiles += ("output_R2_gzip" -> R2)
+      outputFiles += ("output_R1_gzip" -> R1)
+      if (paired) outputFiles += ("output_R2_gzip" -> R2)
 
-    if (!skipTrim || !skipClip) {
       fastqc_R1_after = Fastqc(this, R1, new File(outputDir, R1_name + ".qc.fastqc/"))
       add(fastqc_R1_after)
       addSummarizable(fastqc_R1_after, "fastqc_R1_qc")