opm-common icon indicating copy to clipboard operation
opm-common copied to clipboard

Inconsistent handling of PORV/MULTPV in EDIT section

Open blattms opened this issue 1 year ago • 0 comments

We have a test PORV in FieldPropsTests.cpp with:

EDIT

BOX
  1 10 1 10 4 4 /

MULTPV
  100*4 /

ENDBOX

BOX
  1 10 1 10 3 3 /

PORV
  100*3 /

ENDBOX

which has the expected result, because when getting PORV from the FieldProperties for writing the first time, we will calculate it from PORO, NTG, MULTPV and cell volume.

But if we put PORV before MULTPV, then the multiplier will not be accounted for and the test breaks, see #4023. Due to the non-overlapping boxes this should definitely be a problem.

What if the boxes overlap or are the same? Would order matter here?

blattms avatar Apr 22 '24 08:04 blattms