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] Add dump config in tools/train #1584

Merged
merged 1 commit into from
Jan 10, 2023
Merged

[Fix] Add dump config in tools/train #1584

merged 1 commit into from
Jan 10, 2023

Conversation

zeakey
Copy link

@zeakey zeakey commented Jan 6, 2023

Motivation

Dump config into work_dir.

Modification

# dump config 
cfg.dump(osp.join(cfg.work_dir, osp.basename(args.config)))

@CLAassistant
Copy link

CLAassistant commented Jan 6, 2023 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@Z-Fran Z-Fran self-requested a review January 10, 2023 05:42
@zengyh1900 zengyh1900 merged commit 5a09a5b into open-mmlab:master Jan 10, 2023
@Z-Fran
Copy link
Collaborator

Z-Fran commented Jan 10, 2023

Thanks for your contribution. @zeakey

@Z-Fran Z-Fran changed the title dump config [Fix] Add dump config in tools/train Jan 10, 2023
Z-Fran added a commit that referenced this pull request Jan 10, 2023
@Z-Fran
Revert "dump config (#1584)"
ba120d0 
This reverts commit 5a09a5b.
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@Z-Fran Z-Fran Z-Fran approved these changes

Assignees

@Z-Fran Z-Fran

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@zeakey @CLAassistant @Z-Fran @zengyh1900