|
1 | 1 | =============================================================== |
2 | 2 |
|
| 3 | +Tag name: cam6_4_044 |
| 4 | +Originator(s): mwaxmonsky |
| 5 | +Date: 10/25/2024 |
| 6 | +One-line Summary: Start refactoring of vertical diffusion to be CCPPized |
| 7 | +Github PR URL: https://github.com/ESCOMP/CAM/pull/1176 |
| 8 | + |
| 9 | +Purpose of changes (include the issue number and title text for each relevant GitHub issue): |
| 10 | +Start refactoring of vertical diffusion to be CCPPized and outsources solve phase of non-graft decomp solves to atmospheric physics solver. |
| 11 | + |
| 12 | +Describe any changes made to build system: |
| 13 | +- Adds the to_be_ccppized directory in atmospheric_physics to source search path |
| 14 | + |
| 15 | +Describe any changes made to the namelist: N/A |
| 16 | + |
| 17 | +List any changes to the defaults for the boundary datasets: N/A |
| 18 | + |
| 19 | +Describe any substantial timing or memory changes: N/A |
| 20 | + |
| 21 | +Code reviewed by: |
| 22 | + |
| 23 | +List all files eliminated: |
| 24 | +D src/utils/coords_1d.F90 |
| 25 | +D src/utils/linear_1d_operators.F90 |
| 26 | +- Files were moved to atmospheric_physics |
| 27 | + |
| 28 | +List all files added and what they do: N/A |
| 29 | + |
| 30 | +List all existing files that have been modified, and describe the changes: |
| 31 | +M .gitmodules |
| 32 | +M bld/configure |
| 33 | +M src/atmos_phys |
| 34 | +- Update atmospheric_physics reference and added new atmospheric_physics directory to search path |
| 35 | + |
| 36 | +M cime_config/config_pes.xml |
| 37 | +- Decreasing NTHRDS from 2 to 1 to force single threaded execution as multithreaded execution enables the -smp flag which breaks under OpenMP when running the SE dycore, similar to #1087 |
| 38 | + |
| 39 | +M src/physics/cam/diffusion_solver.F90 |
| 40 | +- Uses new CCPPized interface for decomp that was moved to atmospheric_physics repo to single solve step. |
| 41 | + |
| 42 | +If there were any failures reported from running test_driver.sh on any test |
| 43 | +platform, and checkin with these failures has been OK'd by the gatekeeper, |
| 44 | +then copy the lines from the td.*.status files for the failed tests to the |
| 45 | +appropriate machine below. All failed tests must be justified. |
| 46 | + |
| 47 | +derecho/intel/aux_cam: |
| 48 | + |
| 49 | +ERP_D_Ln9.ne30pg3_ne30pg3_mg17.FLTHIST.derecho_intel.cam-outfrq9s (Overall: DIFF) |
| 50 | +ERP_Ld3.ne30pg3_ne30pg3_mg17.FHISTC_MTt4s.derecho_intel.cam-outfrq1d_aoa (Overall: DIFF) |
| 51 | +SMS_D_Ln9.ne30pg3_ne30pg3_mg17.FMTHIST.derecho_intel.cam-outfrq9s (Overall: DIFF) |
| 52 | +SMS_D_Ln9_P1280x1.ne30pg3_ne30pg3_mg17.FHISTC_MTt1s.derecho_intel.cam-outfrq9s_Leung_dust (Overall: DIFF) |
| 53 | +- expected diffs due to changes in cam7 configuration/tuning |
| 54 | + |
| 55 | +ERP_Ln9.f09_f09_mg17.FCSD_HCO.derecho_intel.cam-outfrq9s (Overall: FAIL) details: |
| 56 | +- pre-existing failure due to HEMCO not having reproducible results issues #1018 and #856 |
| 57 | + |
| 58 | +SMS_D_Ln9.f19_f19_mg17.FXHIST.derecho_intel.cam-outfrq9s_amie (Overall: FAIL) |
| 59 | +SMS_D_Ln9_P1280x1.ne0CONUSne30x8_ne0CONUSne30x8_mt12.FCHIST.derecho_intel.cam-outfrq9s (Overall: FAIL) |
| 60 | +- pre-existing failures due to build-namelist error requiring CLM/CTSM external update. |
| 61 | + |
| 62 | +derecho/nvhpc/aux_cam: |
| 63 | + |
| 64 | +ERS_Ln9_G4-a100-openacc.ne30pg3_ne30pg3_mg17.F2000dev.derecho_nvhpc.cam-outfrq9s_mg3_default (Overall: DIFF) |
| 65 | +- expected diff due to changing seasalt_emis_scale for cam7 |
| 66 | + |
| 67 | +izumi/nag/aux_cam: |
| 68 | + |
| 69 | +DAE.f45_f45_mg37.FHS94.izumi_nag.cam-dae (Overall: FAIL) details: |
| 70 | + - pre-existing failure -- issue #670 |
| 71 | + |
| 72 | +izumi/gnu/aux_cam: |
| 73 | + |
| 74 | +ERP_D_Ln9.ne3pg3_ne3pg3_mg37.FLTHIST.izumi_gnu.cam-outfrq9s (Overall: DIFF) details: |
| 75 | +ERP_Ln9_P24x2.f45_f45_mg37.QPWmaC6.izumi_gnu.cam-outfrq9s_mee_fluxes (Overall: DIFF) details: |
| 76 | +ERS_Ln9_P24x1.mpasa480_mpasa480.F2000climo.izumi_gnu.cam-outfrq9s_mpasa480 (Overall: DIFF) details: |
| 77 | +- new tests are missing baselines |
| 78 | + |
| 79 | +CAM tag used for the baseline comparison tests if different than previous |
| 80 | +tag: |
| 81 | + |
| 82 | +Summarize any changes to answers, i.e., |
| 83 | +- what code configurations: |
| 84 | +- what platforms/compilers: |
| 85 | +- nature of change (roundoff; larger than roundoff but same climate; new |
| 86 | + climate): |
| 87 | +N/A |
| 88 | + |
| 89 | +If bitwise differences were observed, how did you show they were no worse |
| 90 | +than roundoff? |
| 91 | +N/A |
| 92 | + |
| 93 | +If this tag changes climate describe the run(s) done to evaluate the new |
| 94 | +climate in enough detail that it(they) could be reproduced, i.e., |
| 95 | +- source tag (all code used must be in the repository): |
| 96 | +- platform/compilers: |
| 97 | +- configure commandline: |
| 98 | +- build-namelist command (or complete namelist): |
| 99 | +- MSS location of output: |
| 100 | +N/A |
| 101 | + |
| 102 | +MSS location of control simulations used to validate new climate: N/A |
| 103 | + |
| 104 | +URL for AMWG diagnostics output used to validate new climate: N/A |
| 105 | + |
| 106 | +=============================================================== |
| 107 | + |
3 | 108 | Tag name: cam6_4_043 |
4 | 109 | Originator(s): eaton |
5 | 110 | Date: 25 Oct 2024 |
|
0 commit comments