Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

[Fix] Fix local rank #1558

Merged
merged 2 commits into from
Apr 2, 2022
Merged

[Fix] Fix local rank #1558

merged 2 commits into from
Apr 2, 2022

Conversation

kennymckormick
Copy link
Member

No description provided.

@kennymckormick kennymckormick merged commit 6ea00d2 into open-mmlab:master Apr 2, 2022
@dreamerlin dreamerlin mentioned this pull request Apr 6, 2022
Closed
@kennymckormick kennymckormick deleted the fix_local_rank branch May 5, 2022 03:35
@makecent
Copy link
Contributor

makecent commented Mar 23, 2023 •
edited
Loading

@hukkai @kennymckormick According to a recent update in pytorch (10 Feb 2023). Now it prefers dash over underscore ( - rather than _).

usage: train.py [-h] [--work-dir WORK_DIR] [--resume [RESUME]] [--amp] [--no-validate] [--auto-scale-lr] [--seed SEED] [--diff-rank-seed] [--deterministic] [--cfg-options CFG_OPTIONS [CFG_OPTIONS ...]]
                [--launcher {none,pytorch,slurm,mpi}] [--local_rank LOCAL_RANK]
                config                                                                                                                                                                                     
train.py: error: unrecognized arguments: --local-rank=0

Maybe we should change back the argument local_rank to local-rank?

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kennymckormick @makecent