OilfieldToolsNet/opm-simulators
4
0
Fork 0
mirror of https://github.com/OPM/opm-simulators.git synced 2025-02-25 18:55:30 -06:00
Code Issues Packages Projects Releases Wiki Activity

Fixed bug in calculation of gravity contribution to boundary flux in the darcyfluxmodule

Browse Source 
There was a sign mistake and a missing division by cell size.
This commit is contained in:
rube051 2020-07-07 09:16:22 +02:00
parent a0be8f8ed4
commit 4dedbeee19
1 changed files with 7 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
opm/models/common/darcyfluxmodule.hh
View File

@ -373,7 +373,7 @@ protected:
// the distance between the face center and the center of the control volume
DimVector distVecIn(posIn);
distVecIn -= posFace;
Scalar absDist = distVecIn.two_norm();
Scalar absDistSquared = distVecIn.two_norm2();
Scalar gTimesDist = gIn*distVecIn;
for (unsigned phaseIdx=0; phaseIdx < numPhases; phaseIdx++) {
@ -385,15 +385,13 @@ protected:
Evaluation pStatIn = - gTimesDist*rhoIn;
Opm::Valgrind::CheckDefined(pStatIn);
// compute the hydrostatic gradient between the two control volumes (this
// gradient exhibitis the same direction as the vector between the two
// control volume centers and the length (pStaticExterior -
// pStaticInterior)/distanceInteriorToExterior. Note that for the
// boundary, 'pStaticExterior' is zero as the boundary pressure is
// defined on boundary face's integration point...
// compute the hydrostatic gradient between the control volume and face integration
// point (this gradient exhibitis the same direction as the vector between the
// control volume center and face integration point (-distVecIn / absDist) and the
// length: -pStaticInterior) / absDist. Note that the two negatives become + and the
// 1 / (absDist * absDist) -> absDistSquared.
EvalDimVector f(distVecIn);
f *= - pStatIn/absDist;
f *= pStatIn / absDistSquared;
// calculate the final potential gradient
for (unsigned dimIdx = 0; dimIdx < dimWorld; ++dimIdx)