Bugfix/issue 527 mixed met vertical indexing#528
Conversation
|
Actually, this doesn't work. I still get flipped wind speed from StateMet output. Perhaps other files have to be changed as well. |
|
Thanks @laura-hyesung-yang for trying to fix this bug. The problem is that the humidity and pressure are from raw GMAO files rather than the derived wind files. Furthermore, the derived wind is not vertically flipped since it is also used in GEOS-Chem Classic nested run, while the raw GMAO fields are flipped. You are headed toward the correct solution, but need to break The full update is more involved since it needs run directory creation updates for automatic setup of the proper settings in Do you want to work on all this or would you like me to put a fix together. This is a pretty egregious bug and I'm sorry I missed it! |
|
Hi @lizziel, Thank you for your explanation! I'd be very grateful if you could put a fix together. Could I ask for one clarification? I thought the derived wind was top-down because when I examined one of the derived wind files (e.g. May I ask if I am misunderstanding? |
|
Hi @laura-hyesung-yang, I figured out yesterday afternoon while putting in my fix that the derived winds and raw C720 files are both top-down. So no, you are not misunderstand. I was just wrong! The problem is happening because we had only one logical for it met-fields are top-down. But that won't work since the advection met-fields are top-down and the rest of the met-fields are bottom-up for the case of lat-lon GEOS-FP used with C720 GEOS-FP. Does that make sense? In my fix I separated out the winds versus humidity in case there is a future case where there is a mismatch. The derived winds files only contains winds, not the humidity, so I think it makes sense to allow different conventions even if the current files are both top-down. |
|
@lizziel Thanks for the clarification. Yes, it makes sense! My run shows reasonable SPHU and wind speed near the surface now. Thank you so much for your help and time, Lizzie! |
|
Great! I will close this PR since it is superceded by #529 and geoschem/geos-chem#3131 which will go into GCHP 14.7.0. |
Name and Institution (Required)
Name: Laura Yang
Institution: WashU in St. Louis
Describe the update
Updated
src/GCHP_GridComp/GCHPctmEnv_GridCompfile to allow usage of raw met. field combined with processed met field. GCHP.rc file should also include:Prior commit "Commits on Nov 24, 2025 112425" is irrelevant to this fix.
Expected changes
Native met. fields won't be flipped and work properly.
This will fix the bug
#527
If this is a science update, please provide a literature citation.
Related Github Issue
#527
Please link to the corresponding Github(s) issue here. If fixing a bug, there should be an issue describing it with steps to reproduce.