Skip to content

Wellbore volume should be scaled based on the pressure and composition in the wellbore in StandardWell#6686

Draft
GitPaean wants to merge 7 commits intoOPM:masterfrom
GitPaean:wellbore_volume
Draft

Wellbore volume should be scaled based on the pressure and composition in the wellbore in StandardWell#6686
GitPaean wants to merge 7 commits intoOPM:masterfrom
GitPaean:wellbore_volume

Conversation

@GitPaean
Copy link
Copy Markdown
Member

It is something should have been done earlier that the fluids inside the wellbore (although it is a small artificial volume for StandardWell) should consider the effects of the pressure and composition.

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

jenkins build this failure_report please

@GitPaean
Copy link
Copy Markdown
Member Author

It does not look like it causes big regression issue. Will try to make it nice so that it is ready for review.

It was brought up again when working on thermal modeling in the wellbore. It is natural that we need it.

@GitPaean
Copy link
Copy Markdown
Member Author

GitPaean commented Jan 9, 2026

jenkins build this failure_report please

@GitPaean
Copy link
Copy Markdown
Member Author

jenkins build this failure_report please

@GitPaean
Copy link
Copy Markdown
Member Author

jenkins build this failure_report please

@GitPaean
Copy link
Copy Markdown
Member Author

The main regression is the case 02_01_wgrupcon.

image

both the master branch and this PR show some unsteady solution at the problematic stage.

@GitPaean GitPaean force-pushed the wellbore_volume branch 3 times, most recently from 9a85e6a to f4b1d64 Compare January 25, 2026 10:17
@GitPaean
Copy link
Copy Markdown
Member Author

GitPaean commented Jan 25, 2026

@GitPaean
Copy link
Copy Markdown
Member Author

jenkins build this failure_report opm-common=4926 please

@GitPaean
Copy link
Copy Markdown
Member Author

GitPaean commented Jan 28, 2026

jenkins build this failure_report please

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

@GitPaean
Copy link
Copy Markdown
Member Author

GitPaean commented Feb 3, 2026

jenkins build this failure_report please

https://ci.opm-project.org/job/opm-simulators-PR-builder/9493/#showFailuresLink

@GitPaean
Copy link
Copy Markdown
Member Author

GitPaean commented Feb 4, 2026

jenkins build this failure_report please

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

@GitPaean
Copy link
Copy Markdown
Member Author

GitPaean commented Feb 4, 2026

The following four cases did not run through in the new jenkins testing results.

default.compareParallelSim_flow+SPE1CASE2_SOLVENT
default.compareECLFiles_flow+SPE1CASE2_GASWATER_SOLVENT
default.compareECLFiles_flow+SPE1CASE2_SOLVENT_FOAM
default.compareECLFiles_flow+SPE5CASE1

other regression failures are fine.

@GitPaean
Copy link
Copy Markdown
Member Author

GitPaean commented Feb 4, 2026

jenkins build this failure_report please

1 similar comment
@GitPaean
Copy link
Copy Markdown
Member Author

GitPaean commented Feb 5, 2026

jenkins build this failure_report please

@GitPaean
Copy link
Copy Markdown
Member Author

GitPaean commented Feb 5, 2026

The regressions are fine now. https://ci.opm-project.org/job/opm-simulators-PR-builder/9506/

The PR needs some cleaning/refactoring before it is ready.

@GitPaean
Copy link
Copy Markdown
Member Author

GitPaean commented Feb 10, 2026

jenkins build this failure_report please
https://ci.opm-project.org/view/PR%20builders/job/opm-simulators-PR-builder/9519/

@GitPaean
Copy link
Copy Markdown
Member Author

jenkins build this failure_report please

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