Stein Krogstad

icon indicating an email [email protected]

icon company SINTEF ICT icon location Norway

Results 45 comments of Stein Krogstad

Fix oscilations

PR fails to complete simulation for `MOD4_UDQ_ACTIONX`, however I'm not able to reproduce this behaviour locally. Will dig a bit more to try figure out what's going on.

Fix oscilations

Found an issue which _by coincidense_ resulted in failed simulation of `MOD4_UDQ_ACTIONX` for this PR. Will attempt to fix this separately.

Fix oscilations

https://github.com/OPM/opm-simulators/pull/5634 has been merged, so I think this should fix the trouble for `MOD4_UDQ_ACTIONX` for this PR. I think the changes here looks good except the comment above. In my...

Fix oscilations

See #5641

Clean up: remove code in if(false) branch

I have to admit I don't recall this was part of https://github.com/OPM/opm-simulators/pull/4895, but I'm fine with deleting it. In general I'm sceptical to accepting wells with very loose convergence criteria...

Set option maximum-number-of-well-switches default value to 10

jenkins build this please

Set option maximum-number-of-well-switches default value to 10

> (just in case: you need to include failure_report in the trigger for the pdfs to be generated) Argh, yes forgot

Set option maximum-number-of-well-switches default value to 10

jenkins build this failure_report please

Set option maximum-number-of-well-switches default value to 10

> build will fail because #5598 was not yet merged. OK, so build with `opm-simulators=5598`?

Set option maximum-number-of-well-switches default value to 10

> The maximum-number-of-well .. options was added to avoid shutting of wells some time ago. I think it is still needed for some cases to avoid issues. The downside is...