Skip to content

Commit

Permalink
Review changes
Browse files Browse the repository at this point in the history
  • Loading branch information
DanSava committed Oct 1, 2024
1 parent c31edd0 commit 028269b
Showing 1 changed file with 5 additions and 9 deletions.
14 changes: 5 additions & 9 deletions src/ert/config/summary_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -69,15 +69,11 @@ def from_config_dict(cls, config_dict: ConfigDict) -> Optional[SummaryConfig]:
)
time_map = set(refcase.dates) if refcase is not None else None
forward_model = config_dict.get(ConfigKeys.FORWARD_MODEL, [])
names = [step[0] for step in forward_model]
simulation_step_exists = False
for job_name in names:
for name in ["eclipse", "flow"]:
if name in job_name.lower():
simulation_step_exists = True
break
if simulation_step_exists:
break
names = [fm_step[0] for fm_step in forward_model]
simulation_step_exists = any(
any(sim in _name.lower() for sim in ["eclipse", "flow"])
for _name in names
)
if not simulation_step_exists:
ConfigWarning.warn(
"Config contains a SUMMARY key but no forward model steps known to generate a summary file"
Expand Down

0 comments on commit 028269b

Please sign in to comment.