Skip to content

feat: WHP constraint - #3987

Open
tjb-ltk wants to merge 192 commits into
developfrom
feature/byer3/wm_we_whp
Open

tjb-ltk wants to merge 192 commits into
developfrom
feature/byer3/wm_we_whp

Conversation

@tjb-ltk

@tjb-ltk tjb-ltk commented Mar 2, 2026

Copy link
Copy Markdown
Contributor

Wellhead pressure constraint option

tjb-ltk and others added 30 commits September 29, 2025 16:50
…aint class hierachy , 3) Remove all old constraint handling methods/variables, 4) it compiles ...
…t compositionalMultiphaseFlow/soreideWhitson/1D_100cells/1D_benchmark.xml
…d convert compositionalMultiphaseFlow/soreideWhitson/1D_100cells/1D_benchmark.xml"

This reverts commit fa61ab1.
…well initialization vagaries assoiated with useSurfaceConditions=0
@tjb-ltk tjb-ltk added type: feature New feature or request ci: run integrated tests Allows to run the integrated tests in GEOS CI ci: run code coverage enables running of the code coverage CI jobs ci: run device builds Allows to run the device (CUDA/HIP) builds labels Sep 23, 2026
@tjb-ltk tjb-ltk added the flag: requires rebaseline Requires rebaseline branch in integratedTests label Sep 29, 2026
Comment thread src/coreComponents/functions/MultivariableNonuniformTableFunction.cpp Outdated
Comment thread src/coreComponents/functions/MultivariableNonuniformTableFunction.cpp Outdated
Comment thread src/coreComponents/functions/MultivariableNonuniformTableFunction.cpp Outdated
Comment thread src/coreComponents/functions/MultivariableNonuniformTableFunction.cpp Outdated
m_axisStepInv[i],
m_axisPoints[i],
axisLows[i], axisStepInv[i], axisMults[i] );
//std::cout << " axis " << i << " index " << axisIndex << " mul " << m_axisHypercubeMults[i] <<std::endl;

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.

Please do a search through the whole diff for std::cout and similar, unless really needed for future debugging.

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.

done and committed

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 device builds Allows to run the device (CUDA/HIP) builds ci: run integrated tests Allows to run the integrated tests in GEOS CI flag: requires rebaseline Requires rebaseline branch in integratedTests type: feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants