Skip to content

fix: thermal prod well inflow der #4024

Open
tjb-ltk wants to merge 19 commits intodevelopfrom
fix/byer3/thermalWell
Open

fix: thermal prod well inflow der #4024
tjb-ltk wants to merge 19 commits intodevelopfrom
fix/byer3/thermalWell

Conversation

@tjb-ltk
Copy link
Copy Markdown
Contributor

@tjb-ltk tjb-ltk commented Apr 9, 2026

Fix to thermal well inflow derivatives.

@tjb-ltk tjb-ltk self-assigned this Apr 9, 2026
@tjb-ltk tjb-ltk added the ci: run CUDA builds Allows to triggers (costly) CUDA jobs label Apr 9, 2026
@tjb-ltk tjb-ltk requested a review from rrsettgast as a code owner April 9, 2026 19:09
@tjb-ltk tjb-ltk added the ci: run integrated tests Allows to run the integrated tests in GEOS CI label Apr 9, 2026
@tjb-ltk tjb-ltk added flag: no rebaseline Does not require rebaseline ci: run code coverage enables running of the code coverage CI jobs labels Apr 9, 2026
@tjb-ltk tjb-ltk changed the title thermal prod well inflow der fix fix: thermal prod well inflow der Apr 9, 2026
@tjb-ltk tjb-ltk marked this pull request as draft April 9, 2026 22:30
@tjb-ltk tjb-ltk marked this pull request as ready for review April 14, 2026 22:15
Copy link
Copy Markdown
Contributor

@dkachuma dkachuma left a comment

Choose a reason for hiding this comment

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

This reads mush better now

Comment thread src/coreComponents/schema/schema.xsd
Comment on lines +343 to +348
bool processMesh = true;
meshBody.forMeshLevels( [&] ( MeshLevel & mesh )
{
if( !processMesh )
return;
processMesh = false;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
bool processMesh = true;
meshBody.forMeshLevels( [&] ( MeshLevel & mesh )
{
if( !processMesh )
return;
processMesh = false;
MeshLevel & mesh = meshBody.getBaseDiscretization()

Could this be what we want here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Better way...Thanks!

tjb-ltk and others added 4 commits April 23, 2026 09:10
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 ci: run CUDA builds Allows to triggers (costly) CUDA jobs ci: run integrated tests Allows to run the integrated tests in GEOS CI flag: no rebaseline Does not require rebaseline flag: ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants