automl / automl/ConfigurableOptimizer

Refactor `BaseProfile` class

Open
#196 1 comment 0 reactions 0 assignees View on GitHub
cleanup
Dominant language
Python
Stars
12
Forks
2
PR merge metrics
No merged PRs in 30d

Description

1. The "searchspace_str" has to be removed from the initializer of `BaseProfile`. It's unintuitive.
2. The file name `profile_config.py` is also confusing and needs to be updated.
3. Method `configure_extra` has a different signature compared to the other `configure_` methods.
4. `get_name_wandb_run` -> improve the name. The emphasis should be on the experiment name, not the "wandb name". Perhaps it's better to move this method elsewhere (having it here is what raises the need for searchspace_str in the init). The Experiment class seems like a suitable candidate.
5. The configs in `_initialize_train_config_xyz` are better of as config (yaml?) files which are included with the library, which are read and loaded into the `BaseProfile` class.
6. `set_searchspace_config` should has the same function but as `configure_xyz` but a different naming convention. Rename it.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.