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

Adds additional unit-of-measure (dimensionality) specifications #88

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

dorchard
Copy link
Collaborator

@dorchard dorchard commented Dec 12, 2024

  • Give units of intermediates: irhoadz, irhoadzdz:

Analysis from the calculation of these values:

Since we have:

        != unit (kg / m**3) :: rho
        real(dp), intent(in) :: rho(:)
            !! air density at pressure levels
...
        != unit 1 :: adz
        real(dp), intent(in) :: adz(:)
            !! ratio of the pressure level grid height spacing [m] to dz (lowest dz spacing)

then

irhoadz(k) = dtn/(rho(k)*adz(k)) !  Temporary factor for below

implies that irhoadz has unit s / ((kg / m**3) * 1) = s m**3 / kg and

irhoadzdz(k) = irhoadz(k)/dz 

implies that irhoadzdz has unit s m**2 / kg

  • give units of q_sed_flux (mass flux per unit area, as per my e-mailed analysis).

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.

1 participant