Support WECON item 9
jenkins build this opm-simulators=6050 please
jenkins build this opm-simulators=6050 please
I agree with @GitPaean. You need something stronger than a linear array to maintain the inter-well relationships implied by WECON(9).
Suppose we have
WECON
'A' ... 'F1' /
'B' ... 'F2' /
'C' ... 'F3' /
/
If well C shuts in we're supposed to (try to) open well F3, not any one of the wells F1, F2, or F3.
I agree with @GitPaean. You need something stronger than a linear array to maintain the inter-well relationships implied by WECON(9).
Note that the inter-well relationship is taken care of by the simulator. This list gives all wells that the simulator opens at any point due to WECON.
jenkins build this opm-simulators=6050 please
I changed the implementation significantly and I am not yet finished so I will make it draft. Just want some help from Jenkins to sort out the missing parts.
jenkins build this opm-simulators=6050 failure_report please
jenkins build this opm-simulators=6050 failure_report please
jenkins build this opm-simulators=6050 failure_report please
jenkins build this opm-simulators=6050 failure_report please
jenkins build this opm-simulators=6050 failure_report please
jenkins build this opm-simulators=6050 failure_report please
jenkins build this opm-simulators=6050 failure_report please
jenkins build this opm-simulators=6050 please
And furthermore, to qualify as a follow on well, the well needs to be a valid SHUT well when parsing, the master branch does not deal with that yet.
I guess checking this on the parser level is less trivial than checking it on the simulator level when it is activated. Is this an abort simulation condition or just a warning?
Is this an abort simulation condition or just a warning?
I think we can treat it as a warning and disqualify the follow on well.
I guess checking this on the parser level is less trivial than checking it on the simulator level when it is activated.
It is fair to say that. If we want to do at the parsing stage, the well needs to be valid and is SHUT in the schedule when parsing WECON.
And checking at the simulator stage probably allowing more flexibility.
And checking at the simulator stage probably allowing more flexibility. Done.
Thanks for the review!
jenkins build this opm-simulators=6050 failure_report please
jenkins build this opm-simulators=6050 failure_report please
jenkins build this opm-simulators=6050 please
jenkins build this opm-simulators=6050 failure_report please
jenkins build this opm-simulators=6050 failure_report please
jenkins build this opm-simulators=6050 update_data please
jenkins build this opm-simulators=6050 update_data please
jenkins build this opm-simulators=6050 opm-tests=1393 please
@GitPaean Any reasons for not merging this?
jenkins build this opm-simulators=6050 opm-tests=1393 please
jenkins build this opm-simulators=6050 update_data please
jenkins build this opm-simulators=6050 opm-tests=1399 please