Skip to content

add multiphase tracer regression test#6582

Merged
bska merged 2 commits into
OPM:masterfrom
akva2:add_mt_tracer_regtest
Oct 30, 2025
Merged

add multiphase tracer regression test#6582
bska merged 2 commits into
OPM:masterfrom
akva2:add_mt_tracer_regtest

Conversation

@akva2
Copy link
Copy Markdown
Member

@akva2 akva2 commented Oct 30, 2025

Add model associated with #6577 in the regression testing suite.

@akva2 akva2 added the manual:irrelevant This PR is a minor fix and should not appear in the manual label Oct 30, 2025
@akva2
Copy link
Copy Markdown
Member Author

akva2 commented Oct 30, 2025

jenkins build this update_data please

Copy link
Copy Markdown
Member

@bska bska left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Other than a potential restart issue, I think this looks good. I also appreciate converting tabs to spaces.

Comment thread regressionTests.cmake Outdated
@akva2 akva2 force-pushed the add_mt_tracer_regtest branch from 31083cf to 1630e50 Compare October 30, 2025 08:06
@akva2
Copy link
Copy Markdown
Member Author

akva2 commented Oct 30, 2025

jenkins build this update_data please

jenkins4opm pushed a commit to jenkins4opm/opm-tests that referenced this pull request Oct 30, 2025
Reason: PR OPM/opm-simulators#6582

opm-common     = ae78571a9fe54577b0bd52adb46a6dd4ced54fea
opm-grid       = 95f828270c9e03c03b4a7adb7db3928aa79f8b6f
opm-simulators = d3700ebf710c898427290f67899717cf65dc97ec

### Changed Tests ###

  * tracer_multiphase
@akva2
Copy link
Copy Markdown
Member Author

akva2 commented Oct 30, 2025

jenkins build this opm-tests=1421 please

bska added a commit to OPM/opm-tests that referenced this pull request Oct 30, 2025
Copy link
Copy Markdown
Member

@bska bska left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for the updates. This looks good to me now and as the new reference solutions have been installed on the CI system, I'll merge this into master to activate the new regression test case.

@bska bska merged commit 7464f46 into OPM:master Oct 30, 2025
2 checks passed
@akva2 akva2 deleted the add_mt_tracer_regtest branch October 30, 2025 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

manual:irrelevant This PR is a minor fix and should not appear in the manual

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants