Skip to content

feat: Add MGR strategy for thermal single-phase flow with wells and poromechanics#4006

Open
victorapm wants to merge 3 commits intodevelopfrom
feature/paludettomag1/thermalSinglePhasePoromechanicsReservoir
Open

feat: Add MGR strategy for thermal single-phase flow with wells and poromechanics#4006
victorapm wants to merge 3 commits intodevelopfrom
feature/paludettomag1/thermalSinglePhasePoromechanicsReservoir

Conversation

@victorapm
Copy link
Copy Markdown
Contributor

cc @castelletto1 @tjb-ltk @frankfeifan

Would you be to help by adding a regression test? Thanks!

@victorapm victorapm added flag: ready for review ci: run integrated tests Allows to run the integrated tests in GEOS CI ci: run CUDA builds Allows to triggers (costly) CUDA jobs ci: run code coverage enables running of the code coverage CI jobs labels Mar 27, 2026
@victorapm victorapm self-assigned this Mar 27, 2026
@rrsettgast rrsettgast removed ci: run CUDA builds Allows to triggers (costly) CUDA jobs ci: run integrated tests Allows to run the integrated tests in GEOS CI labels Mar 27, 2026
@frankfeifan
Copy link
Copy Markdown
Contributor

cc @castelletto1 @tjb-ltk @frankfeifan

Would you be to help by adding a regression test? Thanks!

Thanks, Victor! I can add one based on the sequential case I made for thermoporomechanics reservoir solver.

@victorapm
Copy link
Copy Markdown
Contributor Author

Thanks, Frank! Feel free to push to this PR

@frankfeifan
Copy link
Copy Markdown
Contributor

frankfeifan commented Mar 28, 2026

Thanks, Frank! Feel free to push to this PR

Hey Victor @victorapm, I've added a fim case for ReservoirThermoPoroElastic and moved both fim and sequential cases to thermoPoromechanics folder. I don't see critical issue in the linear solver for the fim case, and it runs well. The log file is attached.
output_fim.log

However, I found the solutions from the fim and sequential are quite different, e.g., see deltaPressure below (left is fim and right is sequential). And I see the fim case shows stability issue at early time. Maybe it can be mitigated by tuning some parameters in xml or mgr recipe? Please feel free to take a look.

Screenshot 2026-03-28 at 1 09 41 PM

@victorapm
Copy link
Copy Markdown
Contributor Author

Thanks Frank! Maybe it's an issue with the FIM formulation? @tjb-ltk do you know if all the Jacobian components are being computed correctly?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci: run code coverage enables running of the code coverage CI jobs flag: ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants