Open arng40 opened 1 month ago
Attention: Patch coverage is 97.81421%
with 4 lines
in your changes missing coverage. Please review.
Project coverage is 55.81%. Comparing base (
94a0e86
) to head (5e8d0af
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@dkachuma I've updated the PR description, you can have a look of a 2D table representation
@arng40 This PR is not approved. I am moving it off of the queue.
This PR follows the https://github.com/GEOS-DEV/GEOS/pull/2984 that aim to standardize logs. Currently PVT tables are only accessible through
.csv
files. In this PR, logs will indicate which csv files are generated along with their associated pathCSV Generated to inputFiles/compositionalMultiphaseWell/test.xml/fluid1_phaseModel1_PhillipsBrineDensity_table.csv :
In addition to csv files, PVT Tables with dimensions n <= 2 can be written in log by meetings 2 conditions :
logLevel
CSV Generated to inputFiles/compositionalMultiphaseWell/test.xml/fluid1_phaseModel1_PhillipsBrineViscosity_table.csv
An example of 2D table :