Skip to content

Commit 36a5446

Browse files
committed
Remove unnecessary asserts
1 parent 4a1890c commit 36a5446

File tree

2 files changed

+29
-6
lines changed

2 files changed

+29
-6
lines changed

tsdate/approx.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -111,15 +111,15 @@ def sufficient_statistics(a_i, b_i, a_j, b_j, y_ij, mu_ij):
111111
112112
:return: normalizing constant, E[t_i], E[log t_i], E[t_j], E[log t_j]
113113
"""
114-
assert a_i > 0 and b_i >= 0, "Invalid parent parameters"
115-
assert a_j > 0 and b_j >= 0, "Invalid child parameters"
116114
assert y_ij >= 0 and mu_ij > 0, "Invalid edge parameters"
117115

118116
a = a_i + a_j + y_ij
119117
b = a_j
120118
c = a_j + y_ij + 1
121119
t = mu_ij + b_i
122120

121+
assert a > 0 and b > 0 and c > 0, "Invalid local posterior"
122+
123123
log_f, sign_f, da_i, db_i, da_j, db_j = hypergeo._hyp2f1(
124124
a_i, b_i, a_j, b_j, y_ij, mu_ij
125125
)

tsdate/hypergeo.py

Lines changed: 27 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -353,11 +353,34 @@ def _hyp2f1_dlmf1583(a_i, b_i, a_j, b_j, y, mu):
353353
a_i, b_i, a_j, b_j, y, mu
354354
)
355355

356-
if np.abs(f_1 - f_2) < _HYP2F1_TOL:
357-
# TODO: detect a priori if this will occur
358-
raise Invalid2F1("Singular hypergeometric function")
359-
360356
f_0 = max(f_1, f_2)
357+
358+
# 2sum
359+
aa = f_1
360+
bb = -1 * f_0
361+
s = aa + bb
362+
ap = s - bb
363+
bp = s - ap
364+
da = aa - ap
365+
db = bb - bp
366+
t = da + db
367+
print("2sum", s, t)
368+
369+
aa = f_2
370+
bb = -1 * f_0
371+
s = aa + bb
372+
ap = s - bb
373+
bp = s - ap
374+
da = aa - ap
375+
db = bb - bp
376+
t = da + db
377+
print("2sum", s, t)
378+
# /2sum
379+
380+
# if np.abs(f_1 - f_2) < _HYP2F1_TOL:
381+
# # TODO: detect a priori if this will occur
382+
# raise Invalid2F1("Singular hypergeometric function")
383+
361384
f_1 = np.exp(f_1 - f_0) * s_1
362385
f_2 = np.exp(f_2 - f_0) * s_2
363386
f = f_1 + f_2

0 commit comments

Comments
 (0)