[Fix] add Config hyperparameters for clearml backend #1559
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation
This PR allows to have better logging of hyperparameters when using ClearML backend.
Modification
Currently, the clearml viz backend
add_cfg()
function callsTask.connect_configuration())
.As stated in the clearml documentation this will save the cfg file as a blog. This is useful for logging purposes but not much use beyond that.
By saving it with
Task.set_parameters_as_dict()
instead, clearml will be aware of the internal structure of the config dictionary and the hyperparameters in it can be used for visualization and evaluation (i.e. we can compare experiments with respect to their hyperparameters)Checklist