Skip to content

Commit a2725bd

Browse files
LouisaPawuschsmithsg84Copilot
authored
Update model equation for hydraulic conductivity (parflow#734)
Added density and gravitational equation. Without those, the units don't match up. K(p) should be in in LT^-1. Since \bar k is in L^2, k_r is in [-], and \mu is in ML^-1T^-1, we would currently be at M^-1LT, which is why we need g in LT^-2 and \rho in ML^-3. Also, see equation (1) here: Blake, G.R. et al. (2008). Permeability. In: Chesworth, W. (eds) Encyclopedia of Soil Science. Encyclopedia of Earth Sciences Series. Springer, Dordrecht. https://doi.org/10.1007/978-1-4020-3995-9_425 --------- Co-authored-by: Steven Smith <smithsg84@gmail.com> Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Co-authored-by: Steven Smith <smith84@llnl.gov>
1 parent 32fa1d4 commit a2725bd

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

docs/user_manual/models.rst

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,9 +90,13 @@ fluxes). The hydraulic conductivity can be written as,
9090
:label: hydcond
9191
9292
\begin{aligned}
93-
K(p) = \frac{{\bar k}k_r(p)}{\mu}
93+
K(p) = \frac{{\bar k} k_r(p) \rho g}{\mu},
9494
\end{aligned}
9595
96+
where :math:`\bar k` is the intrinsic permeability tensor, :math:`k_r(p)` is
97+
the relative permeability, :math:`\rho` is the density, :math:`g` is the
98+
gravitational acceleration, and :math:`\mu` is the viscosity.
99+
96100
Boundary conditions can be stated as,
97101

98102
.. math::

0 commit comments

Comments
 (0)