X-Git-Url: http://woldlab.caltech.edu/gitweb/?p=samtools.git;a=blobdiff_plain;f=bcftools%2Fem.c;h=b7dfe1a2889e08de37deb725dd7569d963bdab91;hp=f5d2fd99e0e4d94ceb64a3d09cbc931515a49fec;hb=0f906dafb2ad22371a753557562ef95c3034480d;hpb=8a8212c76299ff302cdd41be704d8e0308bd2b1b

diff --git a/bcftools/em.c b/bcftools/em.c
index f5d2fd9..b7dfe1a 100644
--- a/bcftools/em.c
+++ b/bcftools/em.c
@@ -168,7 +168,7 @@ static double lk_ratio_test(int n, int n1, const double *pdg, double f3[3][3])
 // x[5..6]: group1 freq, group2 freq
 // x[7]: 1-degree P-value
 // x[8]: 2-degree P-value
-int bcf_em1(const bcf1_t *b, int n1, int flag, double x[9])
+int bcf_em1(const bcf1_t *b, int n1, int flag, double x[10])
 {
 	double *pdg;
 	int i, n, n2;
@@ -180,7 +180,7 @@ int bcf_em1(const bcf1_t *b, int n1, int flag, double x[9])
 	n = b->n_smpl; n2 = n - n1;
 	pdg = get_pdg3(b);
 	if (pdg == 0) return -1;
-	for (i = 0; i < 9; ++i) x[i] = -1.;
+	for (i = 0; i < 10; ++i) x[i] = -1.; // set to negative
 	{
 		if ((x[0] = est_freq(n, pdg)) < 0.) {
 			free(pdg);
@@ -188,7 +188,7 @@ int bcf_em1(const bcf1_t *b, int n1, int flag, double x[9])
 		}
 		x[0] = freqml(x[0], 0, n, pdg);
 	}
-	if (flag & (0xf<<1|1<<8)) { // estimate the genotype frequency and test HWE
+	if (flag & (0xf<<1|3<<8)) { // estimate the genotype frequency and test HWE
 		double *g = x + 1, f3[3], r;
 		f3[0] = g[0] = (1 - x[0]) * (1 - x[0]);
 		f3[1] = g[1] = 2 * x[0] * (1 - x[0]);
@@ -207,20 +207,24 @@ int bcf_em1(const bcf1_t *b, int n1, int flag, double x[9])
 		x[6] = freqml(x[0], n1, n, pdg);
 	}
 	if ((flag & 1<<7) && n1 > 0 && n1 < n) { // 1-degree P-value
-		double f[3], f3[3][3];
+		double f[3], f3[3][3], tmp;
 		f[0] = x[0]; f[1] = x[5]; f[2] = x[6];
 		for (i = 0; i < 3; ++i)
 			f3[i][0] = (1-f[i])*(1-f[i]), f3[i][1] = 2*f[i]*(1-f[i]), f3[i][2] = f[i]*f[i];
-		x[7] = kf_gammaq(.5, log(lk_ratio_test(n, n1, pdg, f3)));
+		tmp = log(lk_ratio_test(n, n1, pdg, f3));
+		if (tmp < 0) tmp = 0;
+		x[7] = kf_gammaq(.5, tmp);
 	}
-	if ((flag & 1<<8) && n1 > 0 && n1 < n) { // 2-degree P-value
-		double g[3][3];
+	if ((flag & 3<<8) && n1 > 0 && n1 < n) { // 2-degree P-value
+		double g[3][3], tmp;
 		for (i = 0; i < 3; ++i) memcpy(g[i], x + 1, 3 * sizeof(double));
 		for (i = 0; i < ITER_MAX; ++i)
 			if (g3_iter(g[1], pdg, 0, n1) < EPS) break;
 		for (i = 0; i < ITER_MAX; ++i)
 			if (g3_iter(g[2], pdg, n1, n) < EPS) break;
-		x[8] = kf_gammaq(1., log(lk_ratio_test(n, n1, pdg, g)));
+		tmp = log(lk_ratio_test(n, n1, pdg, g));
+		if (tmp < 0) tmp = 0;
+		x[8] = kf_gammaq(1., tmp);
 	}
 	// free
 	free(pdg);