From d4e7b72c2eac3ad7ce31f51d585793639f654e6a Mon Sep 17 00:00:00 2001 From: Peter Verveer Date: Mon, 16 Dec 2024 14:36:49 +0100 Subject: [PATCH] Fix everest batch numbering (#9563) --- src/ert/run_models/everest_run_model.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/ert/run_models/everest_run_model.py b/src/ert/run_models/everest_run_model.py index 8d7f9fdef12..aa28a26ef55 100644 --- a/src/ert/run_models/everest_run_model.py +++ b/src/ert/run_models/everest_run_model.py @@ -682,18 +682,21 @@ def _slug(entity: str) -> str: self._simulator_cache.add_simulation_results( sim_idx, real_id, control_values, objectives, constraints ) - self.batch_id += 1 # Note the negative sign for the objective results. Everest aims to do a # maximization, while the standard practice of minimizing is followed by # ropt. Therefore we will minimize the negative of the objectives: - return EvaluatorResult( + evaluator_result = EvaluatorResult( objectives=-objectives, constraints=constraints, batch_id=self.batch_id, evaluation_ids=sim_ids, ) + self.batch_id += 1 + + return evaluator_result + def send_snapshot_event(self, event: Event, iteration: int) -> None: super().send_snapshot_event(event, iteration) if type(event) in (EESnapshot, EESnapshotUpdate):