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

Units of adv_coefs? #1097

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

theweathermanda
Copy link
Contributor

I've been trying to track units through the dynamics procedure in mpas_atm_time_integration and have found it difficult as several of the units for variables in the Registry are set to unitless when they apparently aren't.

The units for adv_coefs and adv_coefs_3rd are a great example of ambiguous units, which––assuming that they are unitless––leads to terms in the tend_w calculation that seemingly cannot be added owing to differing units. Having dug through the code, I think that the units of these terms should be m^3 since the coefficient values are multiplied by dcEdge**2 and dvEdge in mpas_atm_core.F. However, I am not confident about this, so I've included it as a separate pull request from my other suggested corrections about units.

As a general issue/request, it would be great if someone more knowledgable than me could check the units for the variables in the Registry files because their ambiguity makes it really hard for members of the community to contribute to the model!

I've been trying to track units through the dynamics procedure in mpas_atm_time_integration and have found it difficult as several of the units for variables in the Registry are set to unitless when they apparently aren't. 

The units for adv_coefs and adv_coefs_3rd are a great example of ambiguous units, which––assuming that they are unitless––leads to terms in the tend_w calculation that cannot be added owing to differing units. Having dug through the code, it would seem that the units of these terms must be m^3 since they are multiplied by dcEdge**2 and dvEdge in mpas_atm_core.F. However, I am not confident about this, so I've included it as a separate pull request from my other suggested corrections about units.
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