code review

hammerlab · Nov 12, 2015 · 830f401 · 830f401
1 parent d8562f7
commit 830f401
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/package.json b/package.json
@@ -11,8 +11,7 @@
     "genomics",
     "sequencing",
     "reads",
-    "interactive",
-    "biojs"
+    "interactive"
   ],
   "main": "dist/main/pileup.js",
   "browser": "dist/main/pileup.js",

diff --git a/src/main/PileupTrack.js b/src/main/PileupTrack.js
@@ -89,6 +89,8 @@ function renderPileup(ctx: DataCanvasRenderingContext2D,
     var left = scale(pos + 1),
         right = scale(pos + refLength + 1),
         bottom = top + READ_HEIGHT,
+        // Arrowheads become a distraction as you zoom out and the reads get
+        // shorter. They should never be more than 1/6 the read length.
         arrowSize = Math.min(READ_STRAND_ARROW_WIDTH, (right - left) / 6);
 
     ctx.beginPath();

diff --git a/src/test/PileupCache-test.js b/src/test/PileupCache-test.js
@@ -288,6 +288,7 @@ describe('PileupCache', function() {
   });
 
   it('should compute statistics on a BAM file', function() {
+    this.timeout(5000);
     var bam = new Bam(
         new RemoteFile('/test-data/synth4.tumor.1.4930000-4950000.bam'),
         new RemoteFile('/test-data/synth4.tumor.1.4930000-4950000.bam.bai'));