Skip to content

Comments

New sorption formulation#2

Draft
Manangka wants to merge 12 commits intorefactor_isothermfrom
new_sorption_formulation
Draft

New sorption formulation#2
Manangka wants to merge 12 commits intorefactor_isothermfrom
new_sorption_formulation

Conversation

@Manangka
Copy link
Owner

This PR contains the revised sorption formulation which ensures mass balance.
An autotest is added that verifies that it does

Checklist of items for pull request

  • Replaced section above with description of pull request
  • Closed issue #xxxx
  • Referenced issue or pull request #xxxx
  • Added new test or modified an existing test
  • Ran ruff on new and modified python scripts in .doc, autotests, doc, distribution, pymake, and utils subdirectories.
  • Formatted new and modified Fortran source files with fprettify
  • Added doxygen comments to new and modified procedures
  • Updated meson files, makefiles, and Visual Studio project files for new source files
  • Updated definition files
  • Updated develop.toml with a plain-language description of the bug fix, change, feature; required for changes that may affect users
  • Updated input and output guide
  • Removed checklist items not relevant to this pull request

For additional information see instructions for contributing and instructions for developing.

wpbonelli and others added 12 commits December 22, 2025 10:51
This test uses coordinate_check_method which hasn't been released yet
)

PRT docs and error messages previously failed to distinguish between cells excluded from the model grid with IDOMAIN and cells merely inactive due e.g. to drying. These two concepts are distinct and should not be confused: PRT will not permit a particle to be released into a cell with IDOMAIN = 0 (excluded from the simulation) or -1 (vertical pass through), but it will permit release into an inactive cell. Behavior in this case is determined by factors such as whether the DRAPE option is activated, whether the flow model uses the Newton formulation, and (if so) which DRY_TRACKING_METHOD is configured.
Co-authored-by: Christian Langevin <clange001@icloud.com>
Co-authored-by: Eric Morway <emorway@usgs.gov>
* fix(sfe): fix sfe storage issue

* fix spelling

* fix an assertion...not sure how it was passing before.  Manual inspection confirms test is acting as it should
…change (MODFLOW-ORG#2621)

* - disable synchronization of mover data when not a primary exchange
 made bookkeeping on synchronizing exchange items (hopefully) more transparent
-improved documentation

* - remove unused

* - add note

* - add release note

* - finished merging notes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants