OilfieldToolsNet/ResInsight
4
0
Fork 0
mirror of https://github.com/OPM/ResInsight.git synced 2025-02-11 07:56:08 -06:00
Code Issues Packages Projects Releases Wiki Activity

Remove obsolete assert

Browse Source 
Discussed with @jacobstoren : Multiple branches are handled by the logic, and the assert is probably obsolete. Consider to set isMultiSegmentWell = true when multiple branches are detected.
This commit is contained in:
Magne Sjaastad 2018-01-02 08:20:03 +01:00
parent bdfd3d4e04
commit 40a4222db2
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
ApplicationCode/ReservoirDataModel/RigSimulationWellCenterLineCalculator.cpp
View File

@ -142,7 +142,7 @@ void RigSimulationWellCenterLineCalculator::calculateWellPipeCenterlineFromWellF
const RigWellResultPoint* prevWellResPoint = NULL; const RigWellResultPoint* prevWellResPoint = NULL;
CVF_ASSERT(isMultiSegmentWell || resBranches.size() <= 1); // CVF_ASSERT(isMultiSegmentWell || resBranches.size() <= 1); // TODO : Consider to set isMultiSegmentWell = true;
// The centerline is calculated by adding a point when the pipe enters a cell, // The centerline is calculated by adding a point when the pipe enters a cell,
// and one when the line leaves the cell. // and one when the line leaves the cell.
@ -150,7 +150,6 @@ void RigSimulationWellCenterLineCalculator::calculateWellPipeCenterlineFromWellF
// The currentResultPoint (Cell) and the one we index by the loop variable is the one we calculate the entry point to. // The currentResultPoint (Cell) and the one we index by the loop variable is the one we calculate the entry point to.
// The previous cell is the one we leave, and calculate the "out-point" from // The previous cell is the one we leave, and calculate the "out-point" from
for (size_t brIdx = 0; brIdx < resBranches.size(); brIdx++) for (size_t brIdx = 0; brIdx < resBranches.size(); brIdx++)
{ {