Skip to content

Commit 8a3c1fa

Browse files
Use kwargs for creating workers in es_trainer_lib
\#453 made keyword arguments required for instantiating worker managers. The callsite in es_trainer_lib was never updated, so updating it now to get things working for ES. Given this was not caught in CI, it might be worth having better typing for the worker managers so that this is caught earlier.
1 parent 5eeb719 commit 8a3c1fa

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

compiler_opt/es/es_trainer_lib.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -213,7 +213,7 @@ def train(additional_compilation_flags=(),
213213

214214
with worker_manager_class(
215215
worker_class,
216-
learner_config.total_num_perturbations,
216+
count=learner_config.total_num_perturbations,
217217
worker_kwargs=dict(gin_config=gin.operative_config_str())) as pool:
218218
for _ in range(learner_config.total_steps):
219219
learner.run_step(pool)

0 commit comments

Comments
 (0)