-
Notifications
You must be signed in to change notification settings - Fork 25
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
GCHP mass flux wrong moisture correction #445
Comments
This makes sense to me. 1/(1-sphu) is the same as kg total / kg dry, but it would make sense to convert by instead multiplying by kg dry/kg total. When you change it do you see improvements in comparisons with wind runs? |
Theoretically, this bug fix is reasonable and a trivial change. But the test reminds me of the previous comparison between mass flux and wind. With / without this fix, the advection in mass flux is much less vigorous than in wind. Together with my recent massflux v.s. wind fullchem benchmark showing unreasonble surface aerosol concentration, I think it is necessary to continue the investigation in #342 with help from @sdeastham . |
Using mass fluxes in GCHP is still experimental, even with this update. Do you want this to be merged or wait until more testing is done? |
I think this can be merged first since it is not intercorrelated with other mass flux issues. |
Your name
Yuanjian Zhang
Your affiliation
WashU
What happened? What did you expect to happen?
GCHP/src/GCHP_GridComp/GCHPctmEnv_GridComp/GCHPctmEnv_GridCompMod.F90
Lines 873 to 885 in e499969
Humidity is positive so
1-SPHU
is smaller than 1, sodryMF = MF/(1-SPHU)
is bigger thanMF
, which does not make sense to me. Shouldn't mass flux from GMAO archives be multiplied instead of being divided by1-SPHU
?The text was updated successfully, but these errors were encountered: