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

🚀[FEA]: Unified config handling #91

Open
mnabian opened this issue Aug 11, 2023 · 0 comments
Open

🚀[FEA]: Unified config handling #91

mnabian opened this issue Aug 11, 2023 · 0 comments
Assignees
Labels
1 - On Deck To be worked on next

Comments

@mnabian
Copy link
Collaborator

mnabian commented Aug 11, 2023

Is this a new feature, an improvement, or a change to existing functionality?

Improvement

How would you describe the priority of this feature request

Medium

Please provide a clear description of problem you would like to solve.

Currently different examples use different ways of handling configs. Some use hydra, some use pydantic, and some use hydra but with a lot of hard-coded configs in the training script. We need to unify these and change all of the examples to properly use the hydra configs.

Describe any alternatives you have considered

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1 - On Deck To be worked on next
Projects
None yet
Development

No branches or pull requests

1 participant