Add optional iterations in the group switching algorithm
Default is 1 i.e. it will only check once.
This if first part of https://github.com/OPM/opm-simulators/pull/5607
jenkins build this please
mind the warning(s).
jenkins build this please
This can be merged as is. But we should also factor out the network part and remove any well solve from updateWellStateWithTarget before considering using this feature. I will make another PR where the network part is refactored to a separate call asap.
jenkins build this please
jenkins build this please
Since default is 1 it should be safe to merge this as is. More testing is needed if we want to change the default.
Thanks for the invitation for review. I will review this later today or tomorrow. I am aware that the PR does not change results by default, while I would like to check the code itself.
Hei, the code change looks straightforward, I think this is towards the good directions, which hopefully make sure the well controls are settled for each iteration before we go ahead to assemble.
From the testing side, do you find this helps any specific case and what is your observation, thanks.
jenkins build this please
Thanks for the update. It looks good to me. I am merging it to make easier for people to test.