Bug fix to explicit in-canopy vertical diffusion of tracers#3186
Conversation
|
@gspetro-NOAA We would kindly request a high priority review of this bug fix . We are targeting getting this fix into UWM for UFS-AQMv8 operations, which has plans to start retro runs in May. Dependent on UFSATM PR NOAA-EMC/ufsatm#1091 and ccpp PR ufs-community/ccpp-physics#371 |
|
@iri01 @drnimbusrain Please run the full suite on either Ursa or Hercules, and push the log and test_changes.list. The description has atmos-cubed-sphere but I don't see a PR there or any changes for that, so I've removed it. Please add info on ACS if there is indeed changes there. Once the full testing is complete and the CCPP-Physics and UFSATM repos get approvals from CM's, then this will get onto the commit queue ASAP. |
|
Thank you, @BrianCurtis-NOAA . Yes, I'm running the RT tests on ursa. Will update info soon. |
|
@iri01 @drnimbusrain @grantfirl cpld_debug_sfs_intel case crashes on derecho |
|
@jkbk2004, thank you for letting us know. ursa path: |
|
@gspetro-NOAA @jkbk2004 Yes, our "cpld_debug_sfs_intel" passes on Ursa as shown here in @iri01 RT, not sure why it fails on Derecho, but we do not have access to that system. |
|
I will run the test on Derecho and see if the test will pass on rerun. Derecho can be finicky. 🙄 If not, I'll try to provide more error info. |
Thank you @gspetro-NOAA !! Hope we can move it forward. |
|
@iri01 Could you sync w/develop? |
|
Ok, I am also getting that crash on Derecho in I'm wondering if it might be an MPI issue? The What other information would it be useful for you to see? I can maybe move my whole run directory to a platform where you have access if you let me know where is best. In case others have Derecho access, my run_dir is at |
@gspetro-NOAA: We just synced. |
Thank you @gspetro-NOAA for the additional information. @iri01 has synced our fork/branch with upstream, and is running another RT on Ursa. However, this is strange and doesn't seem related to our most recent canopy changes to UFSATM/ccpp PBL scheme in this PR. Have other recent PRs been similarly tested for this |
|
Testing has completed successfully; leaving a note in sub-PRs. |
|
@gspetro-NOAA OK, saw that UFSATM was also merged, so I reverted .gitmodules and submodule here. Thank you. |
Commit Queue Requirements:
test_changes.listindicates which tests, if any, are changed by this PR. Committest_changes.list, even if it is empty.Description:
Commit Message:
Priority:
Sub component Pull Requests:
UFSWM Blocking Dependencies:
Documentation:
Changes
Regression Test Changes (Please commit test_changes.list):
Updates are expected to change Canopy ON regression test baselines due to removal of unwanted explicit in-canopy diffusion effects on hbpl/kpbl, which alters the overall canopy effect on predicted meteorology and chemistry.
Input data Changes:
Library Changes/Upgrades:
Testing Log: