It turns out that Math.log10() can return 0, which leads to QUALs being set to -0, which is off-spec.
git-svn-id: file:///humgen/gsa-scr1/gsa-engineering/svn_contents/trunk@5415 348d0f76-0448-11de-a6fe-93d51630548a
This commit is contained in:
parent
3596c56602
commit
4baeb5979f
|
|
@ -29,7 +29,7 @@ import org.broadinstitute.sting.commandline.Argument;
|
|||
import org.broadinstitute.sting.commandline.Hidden;
|
||||
|
||||
|
||||
public class UnifiedArgumentCollection {
|
||||
public class UnifiedArgumentCollection {
|
||||
|
||||
// control the various models to be used
|
||||
@Argument(fullName = "genotype_likelihoods_model", shortName = "glm", doc = "Genotype likelihoods calculation model to employ -- SNP is the default option, while DINDEL is also available for calling indels.", required = false)
|
||||
|
|
|
|||
|
|
@ -78,7 +78,7 @@ public class QualityUtils {
|
|||
}
|
||||
|
||||
static public double phredScaleErrorRate(double errorRate) {
|
||||
return -10.0*Math.log10(errorRate);
|
||||
return Math.abs(-10.0*Math.log10(errorRate));
|
||||
}
|
||||
|
||||
static public double lodToPhredScaleErrorRate(double lod) {
|
||||
|
|
|
|||
Loading…
Reference in New Issue