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

[bugfix] Use reasonable timestep if NEXTSTEP in ACTIONX is used.

Browse Source 
The step size might still need to be smaller than suggested to not
simulate beyond the end of the current report step. To ensure this we
now use AdaptiveTimeSimulationTimer::provideTimeStepEstimate which
will limit it and also make sure that subsequent time steps will not
get to small either.
This commit is contained in:
Markus Blatt 2024-03-21 10:18:56 +01:00
parent a7ee48c2d5
commit 98e11bf14e
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
opm/simulators/timestepping/AdaptiveTimeStepping.hpp
View File

@ -387,7 +387,8 @@ std::set<std::string> consistentlyFailingWells(const std::vector<StepReport>& sr
// get current delta t
auto oldValue = suggestedNextTimestep_;
if (tuningUpdater()) {
substepTimer.setCurrentStepLength(suggestedNextTimestep_);
// Use provideTimeStepEstimate to make we sure don't simulate longer than the report step is.
substepTimer.provideTimeStepEstimate(suggestedNextTimestep_);
suggestedNextTimestep_ = oldValue;
}
const double dt = substepTimer.currentStepLength();