Hmm, not good. Fixing the math in PBT resulted in changed MD5s for integration tests that look like significant changes. I am reverting and will report this to Laurent.
This commit is contained in:
parent
65c594afff
commit
53383e82ec
|
|
@ -645,7 +645,7 @@ public class PhaseByTransmission extends RodWalker<HashMap<Byte,Integer>, HashMa
|
||||||
bestChildGenotype.clear();
|
bestChildGenotype.clear();
|
||||||
bestChildGenotype.add(childGenotype.getKey());
|
bestChildGenotype.add(childGenotype.getKey());
|
||||||
}
|
}
|
||||||
else if(MathUtils.compareDoubles(configurationLikelihood, bestConfigurationLikelihood) == 0) {
|
else if(configurationLikelihood == bestConfigurationLikelihood) {
|
||||||
bestFirstParentGenotype.add(firstParentGenotype.getKey());
|
bestFirstParentGenotype.add(firstParentGenotype.getKey());
|
||||||
bestSecondParentGenotype.add(secondParentGenotype.getKey());
|
bestSecondParentGenotype.add(secondParentGenotype.getKey());
|
||||||
bestChildGenotype.add(childGenotype.getKey());
|
bestChildGenotype.add(childGenotype.getKey());
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue