diff --git a/protected/java/src/org/broadinstitute/sting/gatk/walkers/diagnostics/missing/QualifyMissingIntervals.java b/protected/java/src/org/broadinstitute/sting/gatk/walkers/diagnostics/missing/QualifyMissingIntervals.java
index 8cc7bb8f3..9fabd6a37 100644
--- a/protected/java/src/org/broadinstitute/sting/gatk/walkers/diagnostics/missing/QualifyMissingIntervals.java
+++ b/protected/java/src/org/broadinstitute/sting/gatk/walkers/diagnostics/missing/QualifyMissingIntervals.java
@@ -76,10 +76,12 @@ import java.util.List;
*
* - Average Base Quality
* - Average Mapping Quality
+ * - Average Depth
* - GC Content
- * - Position in the target
- * - Coding Sequence / Intron
- * - Length of the uncovered area
+ * - Position in the target (Integer.MIN_VALUE if no overlap)
+ * - Length of the overlapping target (zero if no overlap)
+ * - Coding Sequence / Intron (optional)
+ * - Length of the uncovered interval
*
*
* Input
@@ -89,7 +91,7 @@ import java.util.List;
*
* Output
*
- * GC content calculations per interval.
+ * GC content, distance from the end of the target, coding sequence intersection, mapping and base quality averages and average depth per "missing" interval.
*
*
* Example
@@ -108,12 +110,24 @@ import java.util.List;
@DocumentedGATKFeature( groupName = HelpConstants.DOCS_CAT_QC, extraDocs = {CommandLineGATK.class} )
@By(DataSource.REFERENCE)
public final class QualifyMissingIntervals extends LocusWalker implements NanoSchedulable {
+ /**
+ * A single GATKReport table with the qualifications on why the intervals passed by the -L argument were missing.
+ */
@Output
protected PrintStream out;
+ /**
+ * List of targets used in the experiment. This file will be used to calculate the distance your missing
+ * intervals are to the targets (usually exons). Typically this is your hybrid selection targets file
+ * (e.g. Agilent exome target list)
+ */
@Argument(shortName = "targets", required = true)
public String targetsFile;
+ /**
+ * List of coding sequence intervals (exons) if different from the targets file, to distinguish intervals
+ * that overlap the cds and intervals that don't.
+ */
@Argument(shortName = "cds", required = false)
public String cdsFile = null;
@@ -224,15 +238,18 @@ public final class QualifyMissingIntervals extends LocusWalker
public void onTraversalDone(List> results) {
for (Pair r : results) {
- GenomeLoc interval = r.getFirst();
- Metrics metrics = r.getSecond();
+ final GenomeLoc interval = r.getFirst();
+ final Metrics metrics = r.getSecond();
+ final List overlappingIntervals = target.getOverlapping(interval);
+
simpleReport.addRow(
interval.toString(),
metrics.gccontent(),
metrics.baseQual(),
metrics.mapQual(),
metrics.depth(),
- getPositionInTarget(interval),
+ getPositionInTarget(interval, overlappingIntervals),
+ getTargetSize(overlappingIntervals),
cds.overlaps(interval),
interval.size(),
interpret(metrics, interval)
@@ -242,13 +259,34 @@ public final class QualifyMissingIntervals extends LocusWalker
out.close();
}
- private int getPositionInTarget(GenomeLoc interval) {
- final List hits = target.getOverlapping(interval);
- int result = 0;
- for (GenomeLoc hit : hits) {
- result = interval.getStart() - hit.getStart(); // if there are multiple hits, we'll get the last one.
+ private int getPositionInTarget(final GenomeLoc interval, final List hits) {
+ if (hits.size() > 0) {
+ final GenomeLoc hit = hits.get(0);
+
+ // interval is larger on both ends than the target -- return the maximum distance to either side as a negative number. (min of 2 negative numbers)
+ if (interval.getStart() < hit.getStart() && interval.getStop() > hit.getStop())
+ return Math.min(interval.getStart() - hit.getStart(),
+ interval.getStop() - hit.getStop());
+
+ // interval is a left overlap -- return a negative number representing the distance between the two starts
+ else if (interval.getStart() < hit.getStart())
+ return hit.getStart() - interval.getStart();
+
+ // interval is a right overlap -- return a negative number representing the distance between the two stops
+ else if (interval.getStop() > hit.getStop())
+ return hit.getStop() - interval.getStop();
+
+ // interval is fully contained -- return the smallest distance to the edge of the target (left or right) as a positive number
+ else
+ return Math.min(Math.abs(hit.getStart() - interval.getStart()),
+ Math.abs(hit.getStop() - interval.getStop()));
}
- return result;
+ // if there is no overlapping interval, return int min value.
+ return Integer.MIN_VALUE;
+ }
+
+ private int getTargetSize(final List overlappingIntervals) {
+ return overlappingIntervals.size() > 0 ? overlappingIntervals.get(0).size() : -1;
}
String interpret(final Metrics metrics, final GenomeLoc interval) {