Skip to content

Commit f6d16cd

Browse files
aivanoufacebook-github-bot
authored andcommitted
Remove username from session name (#215)
Summary: Pull Request resolved: #215 The diff makes a default torchx session name as `torchx` instead of `torchx-$username` RFC: #227 Reviewed By: d4l3k Differential Revision: D31348341 fbshipit-source-id: ff50599a629907d888cebccdeab948df2a17938d
1 parent 2d5281e commit f6d16cd

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

torchx/runner/api.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
# This source code is licensed under the BSD-style license found in the
66
# LICENSE file in the root directory of this source tree.
77

8-
import getpass
98
import json
109
import logging
1110
import time
@@ -547,7 +546,7 @@ def get_runner(name: Optional[str] = None, **scheduler_params: Any) -> Runner:
547546
548547
"""
549548
if not name:
550-
name = f"torchx_{getpass.getuser()}"
549+
name = "torchx"
551550

552551
schedulers = get_schedulers(session_name=name, **scheduler_params)
553552
return Runner(name, schedulers)

torchx/runner/test/api_test.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -423,3 +423,7 @@ def test_runner_try_catch(self, _) -> None:
423423

424424
# can call close twice
425425
runner.close()
426+
427+
def test_get_default_runner(self, _) -> None:
428+
runner = get_runner()
429+
self.assertEqual("torchx", runner._name)

0 commit comments

Comments
 (0)