Stein Krogstad

Results 45 comments of Stein Krogstad

I think it's preferable to merge anything in the pipeline before https://github.com/OPM/opm-simulators/pull/5157. I'm also hoping to get in a few improvements prior to https://github.com/OPM/opm-simulators/pull/5157

I have the same impression, I haven't seen anything worrying. I am a bit surprised that it affects this many cases though, but there's probably some effect I haven't thought...

So probably no worries here, but I guess we should make sure we don't miss anything. @GitPaean and @totto82 , could we just devide the 60 failed tests between us...

Looks like there are some issues that needs fixing here. Seems to be missing a `calculateExplicitQuantities` in the potential calculations for STW. Hence, with the PR, there is zero p-drop...

I've started to run the tests for seperate parts of the PR in attempt to get a better overview. I don't know whether the `--local-well-solve-control-switching`-option has ever been run on...

I'm working on this and have gone through all failures that `--local-well-solve-control-switching=true` are contributing to. This was a larger number (175) than anticipated and mainly due to differences in potential...

@GitPaean @totto82 thanks! From what I can see, differences are small compared to e.g., changing the linear solver ;). At some point we discussed having a seperate _well_state_ for the...

> Turning of the local switch control with --local-well-solve-control-switching=false can avoid to get BHP lower than the bhp limit. Maybe it is something interesting to investigate. @steink There are quite...

> This PR with --shut-unsolvable-wells=false restore Norne results Looking into this. Currently a well that has reached it's maximum switching limit is not only prevented from switching control, but also...

@totto82 could you make the following small update: https://github.com/OPM/opm-simulators/commit/aec51705bb01951f87351562213d73f93018f093 ? This will prevent forcing a non-operable well to be open in cases the control mode is forced (prevents well convergence...