Skip to content

Use hybrid compositional flash by default - #7403

Merged
GitPaean merged 1 commit into
OPM:masterfrom
GitPaean:flash_default
Sep 17, 2026
Merged

GitPaean merged 1 commit into
OPM:masterfrom
GitPaean:flash_default

Conversation

@GitPaean

@GitPaean GitPaean commented Sep 14, 2026

Copy link
Copy Markdown
Member

with OPM/opm-common#5245, newton solver will fall back to ssi if not converged.

it is also showing that the hybrid compositional flash solver produces some converged results that ssi failed to do so when working with #7306 .

@GitPaean GitPaean added the manual:irrelevant This PR is a minor fix and should not appear in the manual label Sep 14, 2026
@GitPaean

Copy link
Copy Markdown
Member Author

jenkins build this failure_report please

@GitPaean

Copy link
Copy Markdown
Member Author

This PR allows us to run through some cases without relaxing the flash tolerance, while it will be topic being revisited in the future development following new testing running.

@GitPaean

Copy link
Copy Markdown
Member Author

This PR will wait for the OPM/opm-common#5365.

@GitPaean

GitPaean commented Sep 16, 2026

Copy link
Copy Markdown
Member Author

jenkins build this opm-common=5365 please

https://ci.opm-project.org/job/opm-simulators-PR-builder/10635/

@GitPaean

Copy link
Copy Markdown
Member Author

A few compositional related are merged, let us see how changing the default method will play out. No regression should be expected.

@GitPaean

GitPaean commented Sep 17, 2026

Copy link
Copy Markdown
Member Author

jenkins build this please

https://ci.opm-project.org/job/opm-simulators-PR-builder/10646/

@GitPaean

Copy link
Copy Markdown
Member Author

As expected, there is no regression from the switching of the default flash method. I am marking it as ready for review now and inviting reviewers.

@GitPaean
GitPaean marked this pull request as ready for review September 17, 2026 11:00
@GitPaean
GitPaean requested a review from bska September 17, 2026 11:00

@akva2 akva2 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Seems straight forward.

@GitPaean
GitPaean merged commit 8932a15 into OPM:master Sep 17, 2026
3 checks passed
@GitPaean
GitPaean deleted the flash_default branch September 17, 2026 11:02
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