- Notifications
You must be signed in to change notification settings - Fork 24
Added Hydra support as an option in ArgParse UI #284
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
Conversation
…n template" This reverts commit cc85736.
#::: } else { :::# | ||
def main(): | ||
config = setup_config() | ||
#::: } :::# |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#::: } :::# | |
#::: } :::# |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| ||
| ||
#::: if ((it.argparser == 'fire')) { :::# | ||
from omegaconf import DictConfig |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line can be refactored
| ||
| ||
#::: } else if ((it.argparser == 'hydra')) { :::# | ||
@hydra.main(version_base=None, config_path=".", config_name="config") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do not understand how to pass backend to the config ?
Which backend is user by default ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
By default backend = None
but if we want to send a specific backend then python main.py ++backend='gloo'
Hydra support in Code Generator
Fix #102
This PR is a continuation of python fire PR (#279) and is raised as a draft for testing now. To know more about Hydra as a tool, visit https://hydra.cc
Additional context
What is the purpose of this pull request?