-
Notifications
You must be signed in to change notification settings - Fork 13
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
Integrate pipeline space into run args yaml + new design for defining constant parameter #96
Integrate pipeline space into run args yaml + new design for defining constant parameter #96
Conversation
danrgll
commented
May 16, 2024
•
edited
Loading
edited
- Declarative usage doc changes
- Define pipeline space in run args yaml
- New design for constant parameter (const1: value)
- Adapt search_space_from_yaml code to new functionalities
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.
Looks good, I left some minor commentary which I'm not going to block anything on. Seems things are a lot more concise and much easier to read, nice one!
The one thing I haven't done is render the docs yet (bad internet), I will do so tomorrow and scroll through it. If all is good, I will merge!