Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding a representation of confined groundwater dynamics into Noah MP 4.0.1 #1562

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sshatanawi
Copy link

Description

Sawsan’s Simulation (SawSim): We are adding a representation of confined groundwater to the Noah Multi-Parameterization land surface model (Noah MP; Niu et al., 2011; Yang et al.,2011). We developed a new scheme to simulate groundwater dynamics in a three-dimensional system containing two fixed thickness groundwater layers. The first layer acts as an unconfined layer and the second layer acts as a confined aquifer. The scheme simulates vertical and lateral flow from the upper soil layers through a confining unit to the second layer. Lateral groundwater-river exchange is based on head gradients between the groundwater head and river level. The impact of pumping on groundwater dynamics from any of the layers will be simulated. We adopted MODFLOW principles to solve the groundwater flow equation numerically using a finite difference method with the Preconditioned Conjugate Gradient (PCG) solver using PETSc libraries (Balay et al., 1998).
Resolves #1561

Testcase

The test case for now is based on a synthetic scenario where all parameters are homogeneous (constant values)

@sshatanawi sshatanawi marked this pull request as draft June 24, 2024 23:58
@emkemp emkemp added the Draft label Jul 25, 2024
@dmocko
Copy link
Contributor

dmocko commented Oct 11, 2024

Hi @sshatanawi

This PR has been labeled as a "Draft" because the only file change is the creation the "SawSim" sub-directory.

Do you have more code that you expect to include in this PR? What is the timeline for this code to be added?

Please note that as this a new feature to LIS, we will need a testcase for us to fully test the code before it is accepted:
https://nasa-lis.github.io/LISF/howto_create_lis_testcases/howto_create_lis_testcases.html

Further, you will want to submit your code from a separate branch instead of your master branch. Please see instructions
on how to add a "feature" branch from Step 5a (Section 4.7.4) in our Working with GitHub guide:
https://nasa-lis.github.io/LISF/working_with_github/working_with_github.html

You likely will want to do this from a clean directory, and then copy over your changed files for this PR after you create the feature branch.

Finally, can you please let us know which LIS team member you are working with on this groundwater scheme?

@sshatanawi
Copy link
Author

sshatanawi commented Oct 14, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Adding a confined groundwater dynamic
3 participants