Skip to content

Commit

Permalink
Merge pull request #7 from tfenne/tf_fix_for_fastas_with_comments
Browse files Browse the repository at this point in the history
"Fix for NullPointerException caused by incorrect handling of FASTA...
  • Loading branch information
editasmedicinedev authored Jan 31, 2023
2 parents 7108886 + 46f7d1a commit 7a3456c
Showing 1 changed file with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ object SearchReference extends LazyLogging {
case class RefWindow(chrom: String, start: Int, end: Int, bases: Array[Byte])

/** Trivial class so we can drop the sequence dictionary as fast as possible. */
private[aligner] class NoDictFastaFile(ref: PathToFasta) extends FastaSequenceFile(ref, false) {
private[aligner] class NoDictFastaFile(ref: PathToFasta) extends FastaSequenceFile(ref, true) {
override def findAndLoadSequenceDictionary(ref: PathToFasta): SAMSequenceDictionary = null
}

Expand Down Expand Up @@ -553,7 +553,9 @@ class SearchReference
progress.record(window.chrom, window.start)
}
catch {
case ex: Throwable => logger.error(s"Encountered an exception: $ex")
case ex: Throwable =>
logger.error(s"Encountered an exception: $ex")
ex.printStackTrace()
}
})
}
Expand Down

0 comments on commit 7a3456c

Please sign in to comment.