Skip to content

Commit 5f2facc

Browse files
alexnikulkovfacebook-github-bot
authored andcommitted
set num_processes=1 in MAB test (#602)
Summary: Pull Request resolved: #602 Attempt to fix the test timing out on CircleCI Differential Revision: D33900623 fbshipit-source-id: a0634539412d14021145c454582814669f5308ef
1 parent 09e36a9 commit 5f2facc

2 files changed

Lines changed: 18 additions & 7 deletions

File tree

reagent/mab/simulation.py

Lines changed: 17 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -188,15 +188,25 @@ def multiple_evaluations_bandit_algo(
188188
)
189189
for _ in range(n_bandits)
190190
)
191-
with Pool(num_processes) as pool:
192-
pseudo_regrets = pool.starmap(
193-
partial(
194-
single_evaluation_bandit_algo,
191+
if num_processes == 1:
192+
pseudo_regrets = [
193+
single_evaluation_bandit_algo(
194+
*a,
195195
update_every=update_every,
196196
freeze_scores_btw_updates=freeze_scores_btw_updates,
197-
),
198-
arguments,
199-
)
197+
)
198+
for a in arguments
199+
]
200+
else:
201+
with Pool(num_processes) as pool:
202+
pseudo_regrets = pool.starmap(
203+
partial(
204+
single_evaluation_bandit_algo,
205+
update_every=update_every,
206+
freeze_scores_btw_updates=freeze_scores_btw_updates,
207+
),
208+
arguments,
209+
)
200210
return np.stack(pseudo_regrets).mean(0)
201211

202212

reagent/test/mab/test_mab.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -374,6 +374,7 @@ def test_multiple_evaluations_bandit_algo(self):
374374
max_steps=max_steps,
375375
algo_kwargs={"n_arms": 2},
376376
bandit_kwargs={"probs": torch.Tensor([0.3, 0.5])},
377+
num_processes=1,
377378
)
378379

379380
self.assertIsInstance(regret_trajectory, np.ndarray)

0 commit comments

Comments
 (0)