Skip to content

Commit db77fe6

Browse files
committed
Revert "Eliminate redundant calculations after incrementing"...
...to prepare for merge of branch "release_revisions" which supersedes the implementation This reverts commit 34390ba.
1 parent 34390ba commit db77fe6

1 file changed

Lines changed: 3 additions & 6 deletions

File tree

prefpy/evbwie1.py

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,6 @@ def aggregate(self, rankings, K, epsilon, tot_iters, epsilon_mm, max_iters_em):
9393
pi_h = np.copy(pi_h0)
9494

9595
inner = tot_iters // max_iters_em
96-
isIncremented = False
9796

9897
for g in range(max_iters_em):
9998
#for g in range(max_iters):
@@ -111,11 +110,9 @@ def aggregate(self, rankings, K, epsilon, tot_iters, epsilon_mm, max_iters_em):
111110
z_h1[i][k] = (pi_h[k] * EMMMixPLAggregator.f(x[i], p_h[k])) / denom_sum
112111

113112
# M-Step:
114-
if not isIncremented:
115-
test = (g + 1) * inner + (max_iters_em - g - 1) * (inner + 1)
116-
if test < tot_iters:
117-
inner += 1
118-
isIncremented = True
113+
test = (g + 1) * inner + (max_iters_em - g - 1) * (inner + 1)
114+
if test < tot_iters:
115+
inner += 1
119116
for l in range(inner):
120117
#for l in range(max_iters_mm):
121118
#for l in range(int(g/50) + 5):

0 commit comments

Comments
 (0)