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

GCHP mass flux wrong moisture correction #445

Open
yuanjianz opened this issue Sep 24, 2024 · 4 comments · May be fixed by #456
Open

GCHP mass flux wrong moisture correction #445

yuanjianz opened this issue Sep 24, 2024 · 4 comments · May be fixed by #456
Assignees
Labels
category: Bug Something isn't working
Milestone

Comments

@yuanjianz
Copy link

yuanjianz commented Sep 24, 2024

Your name

Yuanjian Zhang

Your affiliation

WashU

What happened? What did you expect to happen?

if ( correct_mass_flux_for_humidity > 0 ) then
MFX_EXPORT = dble(temp3d_r4) / ( 1.d0 - SPHU0_EXPORT )
else
MFX_EXPORT = dble(temp3d_r4)
endif
call MAPL_GetPointer(IMPORT, temp3d_r4, 'MFYC', RC=STATUS)
_VERIFY(STATUS)
if ( correct_mass_flux_for_humidity > 0 ) then
MFY_EXPORT = dble(temp3d_r4) / ( 1.d0 - SPHU0_EXPORT )
else
MFY_EXPORT = dble(temp3d_r4)
endif

Humidity is positive so 1-SPHU is smaller than 1, so dryMF = MF/(1-SPHU) is bigger than MF, which does not make sense to me. Shouldn't mass flux from GMAO archives be multiplied instead of being divided by 1-SPHU?

@yuanjianz yuanjianz added the category: Bug Something isn't working label Sep 24, 2024
@lizziel
Copy link
Contributor

lizziel commented Sep 25, 2024

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?

@yuanjianz
Copy link
Author

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 .

@lizziel
Copy link
Contributor

lizziel commented Oct 1, 2024

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?

@lizziel lizziel self-assigned this Oct 9, 2024
@yuanjianz
Copy link
Author

I think this can be merged first since it is not intercorrelated with other mass flux issues.

@yuanjianz yuanjianz changed the title GCHP mass flux moisture correction GCHP mass flux wrong moisture correction Oct 30, 2024
@yuanjianz yuanjianz linked a pull request Oct 30, 2024 that will close this issue
@lizziel lizziel added this to the 14.5.1 milestone Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants