From 31fa103e0a5551f98ddca8911db1e3884a91eaf7 Mon Sep 17 00:00:00 2001 From: Alexander Tornede Date: Mon, 19 Feb 2024 10:06:25 +0100 Subject: [PATCH 1/2] Fix bug where the local importance plots were all identical independent of the cost function used (#71) --- deepcave/evaluators/lpi.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/deepcave/evaluators/lpi.py b/deepcave/evaluators/lpi.py index 404a6af5..4abaecc6 100644 --- a/deepcave/evaluators/lpi.py +++ b/deepcave/evaluators/lpi.py @@ -60,7 +60,9 @@ def calculate( self.rs = np.random.RandomState(seed) # Get data - df = self.run.get_encoded_data(budget=budget, specific=True, include_combined_cost=True) + df = self.run.get_encoded_data( + objectives=objectives, budget=budget, specific=True, include_combined_cost=True + ) X = df[self.hp_names].to_numpy() Y = df[COMBINED_COST_NAME].to_numpy() From a0ac677df0a3a3e93f1896dee329229d749661d3 Mon Sep 17 00:00:00 2001 From: Sarah Krebs Date: Wed, 21 Feb 2024 09:55:21 +0100 Subject: [PATCH 2/2] Add changelog entry --- CHANGELOG.md | 1 + 1 file changed, 1 insertion(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index beb3d82d..c860e1bf 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -6,6 +6,7 @@ - Do not disable existing loggers. ## Bug-Fixes +- Fix missing objective specification in LPI evaluator (#71). - Don't convert BOHB runs with status 'running' (consistent with SMAC). - Fix api examples (#68). - Reset inputs to fix error when subsequently selecting runs with different configspaces, objectives or budgets (#106).