-
Notifications
You must be signed in to change notification settings - Fork 114
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adds support for the DUMPCUPL keyword #4276
Conversation
jenkins build this opm-simulators=5620 please |
Okay. In that case I'll set the PR to draft state in order to avoid inadvertently merging the work before the upstream PR is in the master sources. |
00f9f20
to
16dbcc6
Compare
16dbcc6
to
6b29ecc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just the usual nitpicks
Adds support for the RCMASTS keyword.
e200348
to
2ca83a3
Compare
@blattms Thanks for the review. I have added two commits to address the comments. |
jenkins build this opm-simulators=5620 please |
did build with companion PR in opm-simulators, trying without. |
jenkins build this please |
Thanks a lot for the changes and the work in total. |
Builds on #4272 which should be merged first.
Adds support for the DUMPCUPL keyword in reservoir coupling.