-
Notifications
You must be signed in to change notification settings - Fork 663
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
Pleim-Xiu Package & Registry update for RA, RS, LANDUSEF, etc. #2025
Conversation
@coastwx Thanks for adding this. You can package RA and RS, but you cannot package LANDUSEF as other LSMs use it too. When you package something with a particular option, those arrays should only be used by that option alone. |
@coastwx I edited your file to remove landusef from package. |
The regression test results:
|
Sorry Wei. I was in meetings earlier. Perfect.
This is good. Thank you.
Rob
From: weiwangncar ***@***.***>
Sent: Thursday, March 21, 2024 12:57 PM
To: wrf-model/WRF ***@***.***>
Cc: Gilliam, Robert ***@***.***>; Mention ***@***.***>
Subject: Re: [wrf-model/WRF] Pleim-Xiu Package & Registry update for RA, RS, LANDUSEF, etc. (PR #2025)
Caution: This email originated from outside EPA, please exercise additional caution when deciding whether to open attachments or click on provided links.
@coastwx<https://github.com/coastwx> I edited your file to remove landusef from package.
-
Reply to this email directly, view it on GitHub<#2025 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ADVGWWHGATKS2RDLLI5TE4LYZMGODAVCNFSM6AAAAABFBJ55A6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJTGAYDOMBXGA>.
You are receiving this because you were mentioned.Message ID: ***@***.******@***.***>>
|
I see RS also passed to NoahMP |
@dudhia Should we just package RA or should we let it go of this PR altogether? |
An alternative is to name them specially like RA_PX and RS_PX and package those. There is a precedent in that LSM. |
Thanks.
Renaming with the RA_PX and RS_PX would cause a chain of changes in the CMAQ system, MCIP in particular so not preferable on our end. Then potential issues with WRF versions in the future where RS is used for pre-v4.6 and RS_PX v4.6+. We'd probably just have to hope users know how to force in the WRF outputs either by the dynamic or compiled methods unless there is another good way. We generally account for this internally, but a lot of external groups have a lot less knowledge of WRF and tend to run out of the box.
Regards,
Rob
From: dudhia ***@***.***>
Sent: Tuesday, March 26, 2024 1:52 PM
To: wrf-model/WRF ***@***.***>
Cc: Gilliam, Robert ***@***.***>; Mention ***@***.***>
Subject: Re: [wrf-model/WRF] Pleim-Xiu Package & Registry update for RA, RS, LANDUSEF, etc. (PR #2025)
Caution: This email originated from outside EPA, please exercise additional caution when deciding whether to open attachments or click on provided links.
An alternative is to name them specially like RA_PX and RS_PX and package those. There is a precedent in that LSM.
-
Reply to this email directly, view it on GitHub<#2025 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ADVGWWFLQJK2SSHDZPMJB3LY2GYT5AVCNFSM6AAAAABFBJ55A6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRRGEYTANRUHE>.
You are receiving this because you were mentioned.Message ID: ***@***.******@***.***>>
|
Renaming could be confined to the model down to the driver with internal physics names the same as before, but yes the output names would be changed, so we can leave it as is. |
@coastwx Considering the complication of adding these variables to package, can we just not do this for now? |
Sure. Disregard the pull request. I doubt there will be much enthusiasm to change the variable names in the WRF output. I'll discuss with our group and perhaps do this in later versions. It would require testing and the window is almost closed for this cycle. And we've gone many years the way it is now. Will update the PX LSM suggested run configuration document. I may be in touch very soon with an update to this hosted document. |
@coastwx Thanks. I will close this PR and we will see what a future solution might be. Please send us any update of the document, probably before the end of April if possible. |
Key variables for CMAQ AQ modeling from the WRF Pleim-Xiu LSM are not default in WRF outputs when LSM=7
TYPE: bug fix
KEYWORDS: Pleim-Xiu, RA, RS, LANDUSEF, Registry
SOURCE: Robert Gilliam, US EPA
DESCRIPTION OF CHANGES:
RA, RS, LANDUSEF, T2OBS and Q2OBS are not default in WRF output when the Pleim-Xiu LSM package is used.
Solution:
We changed the Registry to add "h" next to variable declarations and added these to the "package pxlsmscheme sf_surface_physics==7" definition.
ISSUE: N/A
LIST OF MODIFIED FILES:
Registry.EM_COMMON
TESTS CONDUCTED:
RELEASE NOTE: Pleim-Xiu updated so all variable used by CMAQ are put in WRF outputs.