Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Add config.yml for global configuration. #60

Closed
wants to merge 0 commits into from
Closed

Add config.yml for global configuration. #60

wants to merge 0 commits into from

Conversation

jiangyuxiaoxiao
Copy link
Collaborator

添加一个全局配置文件来配置bert_genresamplepreprocess_texttrain_msserver的所有全局配置。现在的配置比较混乱,有的是代码写死的,比如server中:

# server.py
hps = utils.get_hparams_from_file("./configs/config.json")
...
_ = utils.load_checkpoint("logs/G_649000.pth", net_g, None, skip_optimizer=True)

有的是通过环境变量指定的,比如train_ms中的RANK
更多的是通过命令行参数指定的。
通过全局配置文件来规范统一所有配置的传入,更详细的可以参考提交的config.yml文件。
具体的配置实现等当前dev分支合并了再提交

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant