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

[Feature] The list type in config does append #1586

Open
BayMaxBHL opened this issue Oct 20, 2024 · 0 comments
Open

[Feature] The list type in config does append #1586

BayMaxBHL opened this issue Oct 20, 2024 · 0 comments

Comments

@BayMaxBHL
Copy link
Contributor

What is the feature?

The config file is really nice in python format, making the config file easy to view.
I also use multiple config merges frequently.
The experiment-config often needs to include datasets-config, schedules-config, and default-runtime-config.
If it is a dictionary-type setting, there is an official tutorial to modify or supplement the specified part.

However, if it is a List, you can only override the setting if you want to supplement it. Totally defeats the purpose of List.
Some common Settings can cause problems with lists.
For example, visualizer vis_backends. Maybe I need to fix TensorboardVisBackend for debugging, and then use WandbVisBackend.
For example, hooks, the official tutorial recommends that customizations be placed in custom_hooks (I later discovered that customizations can also be placed in default_hooks to solve this problem), and different hooks distributed in multiple config cannot be merged into one custom_hooks.

Any other context?

No response

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

1 participant