Skip to content

Move relperm driver#4040

Open
dkachuma wants to merge 44 commits into
developfrom
dkachuma/refactor/rel-perm-driver
Open

Move relperm driver#4040
dkachuma wants to merge 44 commits into
developfrom
dkachuma/refactor/rel-perm-driver

Conversation

@dkachuma

@dkachuma dkachuma commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

Building on the constitutive testing infrastructure established in PR3883, this pull request moves the RelpermDriver use the same framework.

  • Updates the driver to inherit from ConstitutiveDriver rather than directly from TaskBase.
  • Adds saturation driver functions to specify the saturation sweep.
  • Introduces a historicalSaturations handling for cases with hysteresis.
  • Consolidates related input files into inputFiles/constitutiveDriver/, adds new ATS integration tests for various model types (Table, Brooks-Corey, van Genuchten), and provides full Sphinx documentation alongside a working XML example.
  • Adds sphinx documentation for the relative permeability driver.
  • Move testRelPermHysteresis unit test from integrationTests to constitutive/unitTests

@dkachuma dkachuma self-assigned this Apr 24, 2026
@dkachuma dkachuma added type: feature New feature or request type: documentation ci: run CUDA builds Allows to triggers (costly) CUDA jobs flag: requires rebaseline Requires rebaseline branch in integratedTests changes XML input 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 labels May 22, 2026
@dkachuma dkachuma marked this pull request as ready for review May 22, 2026 21:49

Copilot AI left a comment

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@jhuang2601 jhuang2601 mentioned this pull request Jun 9, 2026
@jhuang2601 jhuang2601 requested a review from joshua-white June 11, 2026 00:17
Comment thread BASELINE_NOTES.md Outdated

@joshua-white joshua-white left a comment

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.

Looks great. Nice tutorial too.

@jhuang2601 jhuang2601 requested a review from bd713 as a code owner June 11, 2026 13:43

@bd713 bd713 left a comment

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.

Thanks @dkachuma

Comment thread src/coreComponents/constitutiveDrivers/docs/RelpermDriver.rst Outdated
Comment thread src/coreComponents/constitutiveDrivers/docs/RelpermDriver.rst
Comment thread src/coreComponents/constitutiveDrivers/relativePermeability/RelpermDriver.cpp Outdated

@jafranc jafranc left a comment

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.

Great addition/correction 👍
Thanks.

Made a dep PR for extension/correction for 3-phases.

Comment thread src/coreComponents/constitutiveDrivers/docs/RelpermDriver.rst Outdated
Comment thread src/coreComponents/constitutiveDrivers/docs/RelpermDriver.rst
jafranc and others added 4 commits June 12, 2026 20:13
* adding 3 phase ternary plots

* typo introduced

* Update src/coreComponents/constitutiveDrivers/docs/RelpermDriver.rst

---------

Co-authored-by: Dickson Kachuma <81433670+dkachuma@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changes XML input 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: requires rebaseline Requires rebaseline branch in integratedTests type: documentation type: feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants