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

Support multiple config files via CLI #191

Closed
JSKenyon opened this issue Nov 13, 2023 · 1 comment
Closed

Support multiple config files via CLI #191

JSKenyon opened this issue Nov 13, 2023 · 1 comment

Comments

@JSKenyon
Copy link
Collaborator

It would be useful to be able to specify multiple config files via the CLI. The logic would be that, given cli file1 file2 file3, their config would be merged such that the last specified file has the highest priority. This should be relatively simple as merging config is a very standard operation in OmegaConf.

The motivation for this is that it would make partial config changes less cumbersome i.e. in my use case, I want to tamper with the backend options, but don't want to create dozens of almost identical config files. With this change, the small handful of backend options I want to tamper with could be placed in a separate file and then simply added via the cli.

@o-smirnov
Copy link
Member

An excellent idea. Works on issue-191 now, will issue a PR shortly.

o-smirnov added a commit that referenced this issue Feb 12, 2024
fixes #191, and cleans up recipe/config loading code
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

No branches or pull requests

2 participants