Skip to content

Commit c2a4229

Browse files
authored
Fix for multigpu runs with docker executor (#134)
* Fix for multigpu runs with docker executor Signed-off-by: Igor Gitman <[email protected]> * Style fix Signed-off-by: Igor Gitman <[email protected]> --------- Signed-off-by: Igor Gitman <[email protected]>
1 parent 8b71365 commit c2a4229

File tree

1 file changed

+1
-1
lines changed
  • src/nemo_run/run/torchx_backend/schedulers

1 file changed

+1
-1
lines changed

src/nemo_run/run/torchx_backend/schedulers/docker.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ def _submit_dryrun(self, app: AppDef, cfg: Executor) -> AppDryRunInfo[DockerJobR
8888
cmd = [role.entrypoint] + role.args
8989
containers.append(
9090
DockerContainer(
91-
name=f"{executor.experiment_id}_{role.name}",
91+
name=f"{executor.experiment_id}_{role.name}".replace("_", "-"),
9292
command=cmd,
9393
executor=_current_executor,
9494
extra_env=role.env,

0 commit comments

Comments
 (0)