Skip to content

Commit b0f6588

Browse files
committed
Bugfix to nimble aligner step
1 parent 915b124 commit b0f6588

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

singlecell/src/org/labkey/singlecell/SingleCellController.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -325,6 +325,10 @@ private static void validateBarcodes(List<Map<String, Object>> readsetRows, Cont
325325
if (rs.get("barcode5") != null) {
326326
uniqueBarcodeNames.add((String.valueOf(rs.get("barcode5"))));
327327
}
328+
329+
if (rs.get("barcode3") != null) {
330+
uniqueBarcodeNames.add((String.valueOf(rs.get("barcode3"))));
331+
}
328332
});
329333

330334
if (!uniqueBarcodeNames.isEmpty())

singlecell/src/org/labkey/singlecell/run/NimbleAligner.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -79,20 +79,20 @@ public AlignmentOutput performAlignment(Readset rs, List<File> inputFastqs1, @Nu
7979
{
8080
File localBam = new File(outputDirectory, basename + ".cellranger.bam");
8181
File localBamIdx = new File(localBam.getPath() + ".bai");
82-
AlignmentOutput output;
82+
AlignmentOutputImpl output = new AlignmentOutputImpl();;
8383

8484
if (localBam.exists() && localBamIdx.exists())
8585
{
8686
getPipelineCtx().getLogger().info("Existing BAM found, re-using: " + localBam.getPath());
87-
output = new AlignmentOutputImpl();
8887
}
8988
else
9089
{
9190
getPipelineCtx().getLogger().info("Running cellranger");
92-
output = super.performAlignment(rs, inputFastqs1, inputFastqs2, outputDirectory, referenceGenome, basename, readGroupId, platformUnit);
91+
AlignmentOutput crOutput = super.performAlignment(rs, inputFastqs1, inputFastqs2, outputDirectory, referenceGenome, basename, readGroupId, platformUnit);
9392

9493
// Remove all the normal 10x outputs:
95-
output.getSequenceOutputs().clear();
94+
output.addCommandsExecuted(crOutput.getCommandsExecuted());
95+
output.addIntermediateFiles(crOutput.getIntermediateFiles());
9696

9797
// Remove the whole 10x folder:
9898
output.addIntermediateFile(output.getBAM().getParentFile().getParentFile());

0 commit comments

Comments
 (0)