Pass on current well state to (implicit) potential calculations

[steink]

Implicit potential calculations makes a copy of the well-state and use that for initial guess. However, WellTesting also deals with a copy of the well-state, so when called from there we need to pass that copy as an argument (the currently used actual well-state may have nonsense values at this stage and occasionally lead to convergence issues when used as initial guess)

[steink]

jenkins build this failure_report please

[GitPaean]

Thanks for the contribution. I will merge it after another jenkins test.

[GitPaean]

jenkins build this please

[GitPaean]

the warning does not look like related to this PR,

pointer used after ‘void operator delete(void*, std::size_t)’ [-Wuse-after-free] 159 | void operator ++ () { ++ refCount(); } | ^~ In member function ‘void ALUGrid::ALUMemoryProvider::GeometryImpl<0, 0, Dune::hexa> >]’, inlined from ‘void ALUGrid::SharedPointer::removeObject() [with ObjectImp = Dune::MyALUGridGeometryImplementation<3>::GeometryImpl<0, 0, Dune::hexa>]’ at /usr/include/dune/alugrid/common/memory.hh:230:32, inlined from ‘ALUGrid::SharedPointer::~SharedPointer() [with ObjectImp = Dune::MyALUGridGeometryImplementation<3>::GeometryImpl<0, 0, Dune::hexa>]’ at /usr/include/dune/alugrid/common/memory.hh:204:19, inlined from ‘Dune::ALU3dGridGeometry<0, 3, const Dune::ALU3dGrid<3, 3, Dune::hexa, Dune::ALUGridMPIComm> >::~ALU3dGridGeometry()’ at /usr/include/dune/alugrid/3d/geometry.hh:630:9, inlined from ‘Dune::Geometry<0, 3, const Dune::ALU3dGrid<3, 3, Dune::hexa, Dune::ALUGridMPIComm>, Dune::ALU3dGridGeometry>::~Geometry()’ at /usr/include/dune/grid/common/geometry.hh:70:9, inlined from ‘Opm::FvBaseProblem::FvBaseProblem(Simulator&) [with TypeTag = Opm::Properties::TTag::FlowProblemAlugrid]’ at /var/lib/jenkins/workspace/opm-simulators-PR-builder/opm/models/discretization/common/fvbaseproblem.hh:132:89:

[akva2]

it is not.