Skip to content
Snippets Groups Projects
Commit 10bb5fd2 authored by Sven Michael Lechner's avatar Sven Michael Lechner
Browse files

Merge branch 'fix_experiment_group_naming' into 'main'

Removing points from experiment group names to fix model saving and loading

See merge request !20
parents 0e8c7aa6 0d592e76
Branches
No related tags found
1 merge request!20Removing points from experiment group names to fix model saving and loading
...@@ -118,7 +118,7 @@ def _group_resolver(cfg: DictConfig, group_by: DictConfig): ...@@ -118,7 +118,7 @@ def _group_resolver(cfg: DictConfig, group_by: DictConfig):
value = value.get(key, "") value = value.get(key, "")
values.append(value) values.append(value)
# concatenate and return # concatenate and return
return "_".join(values) return "_".join(values).replace(".", "")
def preprocess_config(cfg: DictConfig): def preprocess_config(cfg: DictConfig):
......
...@@ -32,7 +32,9 @@ class ConfigHelpersTest(unittest.TestCase): ...@@ -32,7 +32,9 @@ class ConfigHelpersTest(unittest.TestCase):
"controller": { "controller": {
"name": "swarm", "name": "swarm",
"scheduler": {"name": "max_battery"}, "scheduler": {"name": "max_battery"},
"adaptive_threshold_fn": {"name": "static"}, "adaptive_threshold_fn": {
"name": "sta.tic"
}, # check that points are removed
}, },
"group_by": { "group_by": {
"controller": [ "controller": [
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment