These 2 changes improve runtime performance almost as much as Ryan's previous attempt (with ID-based comparisons): * Don't unnecessarily overload Allele.getBases() in the Haplotype class. * Haplotype.getBases() was calling clone() on the byte array. * Added a constructor to Allele (and Haplotype) that takes in an Allele as input. * It makes a copy of he given allele without having to go through the validation of the bases (since the Allele has already been validated). * Rev'ed the variant jar accordingly. For the reviewer: all tests passed before rebasing, so this should be good to go as far as correctness. |
||
|---|---|---|
| .. | ||
| com.google.code.caliper | ||
| com.google.code.cofoja | ||
| edu.mit.broad | ||
| gov.nist | ||
| net.sf | ||
| net.sf.gridscheduler | ||
| net.sf.snpeff | ||
| org.broad | ||
| org.broadinstitute | ||