Skip to content

Commit

Permalink
Remove duplicate no_ssh_check arg (#831)
Browse files Browse the repository at this point in the history
* Remove duplicate no_ssh_check arg

* Update NeoXArgs docs automatically

* Update NeoXArgs docs automatically

---------

Co-authored-by: github-actions <[email protected]>
Co-authored-by: Stella Biderman <[email protected]>
  • Loading branch information
3 people authored Mar 14, 2023
1 parent dd5a53d commit 6534f19
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 6 deletions.
2 changes: 1 addition & 1 deletion configs/neox_arguments.md
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ Logging Arguments

- **git_hash**: str

Default = 09fc35d
Default = e0587af

current git hash of repository

Expand Down
5 changes: 0 additions & 5 deletions megatron/neox_arguments/deepspeed_args.py
Original file line number Diff line number Diff line change
Expand Up @@ -216,8 +216,3 @@ class NeoXArgsDeepspeedRunner(NeoXArgsTemplate):
"""
Adds a `--comment` to the DeepSpeed launch command. In DeeperSpeed this is passed on to the SlurmLauncher as well. Sometime necessary for cluster rules, or so I've heard.
"""

no_ssh_check: bool = False
"""
If `True` and running with multiple nodes, then DeepSpeedd doesn't conduct a check to ensure the head node is reachable with ssh.
"""

0 comments on commit 6534f19

Please sign in to comment.