Remove unnecessary get_costs function, rename example log files

Author: Sarah Krebs

CHANGELOG.md

@@ -11,6 +11,7 @@
 - Allow to load multi-objective SMAC3v2 and add example (#69).
 - Allow to load runs with multiple seeds and add examples (#70).
 - Correct incumbent calculation when single objective should be maximized.
+- Correct range of configuration cube slider for number of configs.
 - Do not disable existing loggers.
 - Update author email.
 - Add exit button which first deletes running jobs and then terminates DeepCave.

deepcave/plugins/summary/configurations.py

@@ -213,17 +213,15 @@ def process(run, inputs) -> Dict[str, Any]:  # type: ignore
                 seeds_evaluated = 0
                 cost = []
                 for seed in seeds:
-                    try:
-                        cost.append(
-                            run.get_costs(
-                                config_id=selected_config_id,
-                                budget=budget,
-                                seed=seed,
-                                statuses=[Status.SUCCESS],
-                            )[seed][objective_id]
-                        )
+                    all_costs = run.get_all_costs(
+                        budget=budget,
+                        seed=seed,
+                        statuses=[Status.SUCCESS],
+                    )
+                    if selected_config_id in all_costs:
+                        cost.append(all_costs[selected_config_id][seed][objective_id])
                         seeds_evaluated += 1
-                    except Exception:
+                    else:
                         continue
 
                 # Add table data

deepcave/runs/__init__.py

@@ -722,7 +722,7 @@ def get_avg_costs(
         objectives = self.get_objectives()
 
         # Budget might not be evaluated
-        config_costs = self.get_costs(config_id, budget, statuses=statuses)
+        config_costs = self.get_all_costs(budget=budget, statuses=statuses)[config_id]
 
         avg_costs, std_costs = [], []
         for idx in range(len(objectives)):
@@ -731,62 +731,6 @@ def get_avg_costs(
             std_costs.append(float(np.std(costs)))
         return avg_costs, std_costs
 
-    def get_costs(
-        self,
-        config_id: int,
-        budget: Optional[Union[int, float]] = None,
-        seed: Optional[int] = None,
-        statuses: Optional[Union[Status, List[Status]]] = None,
-    ) -> Dict[int, List[float]]:
-        """
-        Return the costs of a configuration.
-
-        Optionally, only configurations which were evaluated on the passed budget, seed, and stati
-        are considered.
-
-        In case of multi-objective, multiple costs are returned in the form of a list.
-
-        Parameters
-        ----------
-        config_id : int
-            Configuration id to get the costs for.
-        budget : Optional[Union[int, float]]
-            Budget to get the costs from the configuration id for. If budget is
-            None, the highest budget is chosen. By default None.
-        seed : Optional[int], optional
-            Seed to get the costs from the configuration id for. If no seed is
-            given, all seeds are considered. By default None.
-        statuses : Optional[Union[Status, List[Status]]]
-            Only selected stati are considered. If no status is given, all stati are considered.
-            By default None.
-
-        Returns
-        -------
-        Dict[int, List[float]]
-            Seeds with their corresponding list of costs for the associated configuration.
-
-        Raises
-        ------
-        ValueError
-            If the configuration id is not found.
-        RuntimeError
-            If the budget was not evaluated for the passed config id.
-        """
-        if budget is None:
-            budget = self.get_highest_budget()
-
-        if config_id not in self.configs:
-            raise ValueError("Configuration id was not found.")
-        costs = self.get_all_costs(budget=budget, seed=seed, statuses=statuses)
-        if config_id not in costs:
-            if seed is not None:
-                raise RuntimeError(
-                    f"Budget {budget} with seed {seed} was not evaluated for config id {config_id}."
-                )
-            else:
-                raise RuntimeError(f"Budget {budget} was not evaluated for config id {config_id}.")
-        return costs[config_id]
-
     def get_all_costs(
         self,
         budget: Optional[Union[int, float]] = None,