Closed Neeratyoy closed 7 months ago
The directories were restructured. Needs to be updated with the master
.
The directories were restructured. Needs to be updated with the
master
.
Done.
Looks good!
There is only one issue with the tests cases for the wrong YAML inputs where I think the test is not being checked correctly.
Overall, it looks quite comprehensive with what looks like adequate documentation! Some other comments I have left as notes for future improvements.
One more such point is, can we now generate a yaml from a pipeline_space
or SearchSpace
and save it along with the files we write for a NePS run?
If so, let us add that too @danrgll @TarekAbouChakra.
@TarekAbouChakra @danrgll Please feel free to coordinate and monitor merges during December. :)
Enabled initialization of search_space via a YAML file. You can find an example in neps_examples/basic_usage/defining_search_space. For more information, refer to docs/pipeline_space.md. Supported Parameter Types: Int, Float, Categorical, Constant