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

Workspace default templates and templates as name-strings or paths in experiment config #6

Closed
marshrossney opened this issue May 5, 2022 · 0 comments · Fixed by #3
Milestone

Comments

@marshrossney
Copy link
Owner

At the moment an experiment config has to have a template field which maps to the Template dataclass containing attributes template, checkout and directory which mirror argument names in cookiecutter.main.cookiecutter.

However, checkout and directory are not required and are not even used if the template is a filepath. This leads to configuration files that look like

name:
    template:
        template: /path/to/template

which is annoying and confusing. If the template is one that has been added to the named templates file then that should also be accepted in the experiment config as template: name.

Also, .pyrex_workspace.yaml should contain a workspace-wide default template to save space in the experiments config file.

@marshrossney marshrossney added this to the Version 1.0 milestone May 5, 2022
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 a pull request may close this issue.

1 participant