-
Notifications
You must be signed in to change notification settings - Fork 123
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
Ensure MSUMLINS and MSUMNEWT is updated. #6039
Ensure MSUMLINS and MSUMNEWT is updated. #6039
Conversation
This makes the summary output correct for these cumulative quantities also at substeps, not just report steps.
jenkins build this please |
jenkins build this failure_report please |
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.
Thanks! This does indeed solve the problem. I don't have an alternative suggestion at the minute so I'm going to update the pertinent reference solutions and then merge this into the master branch.
jenkins build this update_data please |
Reason: PR OPM/opm-simulators#6039 opm-common = a6e4b524da7ee6edf3d5ac07540fcb9d3fbaf4b2 opm-grid = 8b67f04151edb2aaa07d9e5796f9ce2d28f2c4b5 opm-simulators = eea38f9309f3c92cc3060fa5603cece06a239d91 ### Changed Tests ### * network_standard * msw_model_1 * base_model_1 * faults_model_1 * udt_1d-01b * udt_1d-01 * udt_1d-02 * udt_1d-03 * norne_reperf * spe1case2_krnum
jenkins build this opm-tests=1300 please |
Automatic Reference Data Update for PR OPM/opm-simulators#6039
The new reference solutions have been installed on the CI system. Merging this now to bring in the fix. |
This makes the summary output correct for these cumulative quantities also at substeps, not just report steps.