OPM / ResInsight

3D viewer and post processing of reservoir models
http://resinsight.org/
GNU General Public License v3.0
175 stars 92 forks source link

Elastic properties : Graceful handling of missing data #6122

Closed magnesj closed 3 years ago

magnesj commented 4 years ago

If any data is missing for a combination of Field, Formation, Facies, return a undefined value instead of return from the function in RimElasticPropertiesCurve::performDataExtraction. If we return here, no data for any of the elastic curves will be displayed.

            RiaLogging::error( QString( "Missing elastic properties. Field='%1', formation='%2', facies='%3'" )
                                   .arg( fieldName )
                                   .arg( formationName )
                                   .arg( faciesName ) );

            double val = std::numeric_limits<double>::infinity();
            values.push_back( val );
        }
kriben commented 4 years ago

Hhmm, this was intentional: missing data will lead to and incomplete frk file. But we can allow the creation of the curve, but stop the export instead?

magnesj commented 4 years ago

Yes, block export. Te user must be able to see where the missing data is located.

hhgs commented 3 years ago

The csv-fil normally contains some trailing ";". Now, this result in error with message "Facies properties import failed:'Incomplete data on line 2:/..." This should be handled by ResInsight.