equinor / fmu-editor

5 stars 2 forks source link

[FEATURE] Lint Eclipse `.DATA` files with `NOSIM` keyword #69

Open mferrera opened 1 year ago

mferrera commented 1 year ago

Eclipse's NOSIM keyword can be inserted into the RUNSPEC section of a .DATA configuration file for linting. This keyword will have Eclipse load the configuration file(s) and associated data but without timestepping through the simulation.

The Error summary looks like this (may or may not be worth grabbing on its own)

 Error summary
 Comments               1
 Warnings              15
 Problems               0
 Errors                 1

Errors and warnings have a format that is easily parsed from the output, prefaced by @

 @--WARNING  AT TIME        0.0   DAYS    ( 1-JAN-0):
 @           SPURIOUS DATA BEFORE TITLE    KEYWORD
 @           /

These warnings and errors are output in sequence after a keyword is read, e.g.

     2 READING INVALID

 @--  ERROR  AT TIME        0.0   DAYS    ( 1-JAN-0):
 @           UNRECOGNISED KEYWORD INVALID  IN RUNSPEC DATA
mferrera commented 1 year ago

It was suggested that OPM Flow catches things Eclipse does not, but the two together may be most complete.

There may be a need for special handling of outliers (i.e. consistently flagged issues that aren't issues)