Started by an SCM change236b89dc55c506ad16e5a3c8eeab762fe3156a8d1e1b89dc55c506ad16e5a3c8eeab762fe3156a8d1e1refs/remotes/origin/clubb_silhs_develb89dc55c506ad16e5a3c8eeab762fe3156a8d1e1b89dc55c506ad16e5a3c8eeab762fe3156a8d1e1refs/remotes/origin/clubb_silhs_develb89dc55c506ad16e5a3c8eeab762fe3156a8d1e1b89dc55c506ad16e5a3c8eeab762fe3156a8d1e1refs/remotes/origin/clubb_silhs_develhttps://github.com/larson-group/cam.githttp://carson.math.uwm.edu/jenkins/blue/organizations/jenkins/cam_BFB_coarse_res_gfortran_test/detail/cam_BFB_coarse_res_gfortran_test/236/artifactshttp://carson.math.uwm.edu/jenkins/blue/organizations/jenkins/cam_BFB_coarse_res_gfortran_test/detail/cam_BFB_coarse_res_gfortran_test/236/changeshttp://carson.math.uwm.edu/jenkins/blue/organizations/jenkins/cam_BFB_coarse_res_gfortran_test/detail/cam_BFB_coarse_res_gfortran_test/236/http://carson.math.uwm.edu/jenkins/blue/organizations/jenkins/cam_BFB_coarse_res_gfortran_test/detail/cam_BFB_coarse_res_gfortran_test/236/testsfalseUpdate sys_admin RepoRun Python Testfalse#23623635773020017cam_BFB_coarse_res_gfortran_test #236236false2363926SUCCESS1779870429255http://carson.math.uwm.edu/jenkins/job/cam_BFB_coarse_res_gfortran_test/236/advance_wp2_wp3_module.F90advance_xp2_xpyp_module.F90advance_xp3_module.F90advance_clubb_core_module.F90advance_xm_wpxp_module.F90stats_clubb_utilities.F90advance_helper_module.F9072b3ab76bcae108c36fc6d3130c7c9c4539bfc281779832578000http://carson.math.uwm.edu/jenkins/user/noreplynoreplynoreply@github.comClubb organization part2 (#1304) A number of fields were being calculated in `advance_clubb_core` (`wp2_zt` `wp3_on_wp2` `wp3_on_wp2_zt` `Skw_zm` `Skw_zt) then passed into other routines, however, all these fields depend on `wp2` and were not being updated after `wp2` was being updated by `calc_sfc_varnce`. This commit fixes that by moving where these fields are calculated - instead of computing in `advance_clubb_core` with the incoming `wp2`, we now calculate them "on-demand" where they are needed using the most recent value of `wp2`. Using our default flag set, there are no numerical differences caused by this change set, only a few stats differences. There are real differences that occur when `l_upwind_xpyp_ta = .false.` (a clubb_config_flag value) or when `l_explicit_turbulent_adv_xpyp = .true.` (a parameter in model_flags.F90), but these are small differences and do not visually affect plots for our stable cases. Changing the order of the `advance_` routines would also cause this to be BIT_CHANGING, since that also affects the update path for `wp2` - now these values that got moved would be using the newest `wp2` instead of being "stale", also causing small (but real) differences. * Calculating wp2_zt and wp3_on_wp2 where needed, rather than passing those values around through advance_clubb_core. This is BIT_CHANGING for a handful of stats only variables be default: wp3_on_wp2, wp3_on_wp2_zt, wp3_on_wp2_cfl_num, term_wprtp2_explicit, term_wprtpthlp_explicit, and term_wpthlp2_explicit. Some Skw_ variables when using lmm_stepping, and has actual differences (not just in stats output) when swapping the order of the advance routines * Moving things around, this seems to all be BFB except for the Skw_ movements, which is BIT_CHANGING if the advance_ order changes * Fixing python API * Reverting some BFB changes 2026-05-26 16:56:18 -050072b3ab76bcae108c36fc6d3130c7c9c4539bfc28Clubb organization part2 (#1304)editadvance_helper_module.F90editadvance_xp3_module.F90editadvance_xm_wpxp_module.F90editadvance_xp2_xpyp_module.F90editstats_clubb_utilities.F90editadvance_wp2_wp3_module.F90editadvance_clubb_core_module.F90githttp://carson.math.uwm.edu/jenkins/user/noreplynoreplynoreplyfalseStarted by an SCM change2352227d41a282c517c62373666a5531dc820ed23602227d41a282c517c62373666a5531dc820ed2360refs/remotes/origin/clubb_silhs_devel2227d41a282c517c62373666a5531dc820ed23602227d41a282c517c62373666a5531dc820ed2360refs/remotes/origin/clubb_silhs_devel2227d41a282c517c62373666a5531dc820ed23602227d41a282c517c62373666a5531dc820ed2360refs/remotes/origin/clubb_silhs_develhttps://github.com/larson-group/cam.gitfalse#23535489713020017cam_BFB_coarse_res_gfortran_test #235235false2353766SUCCESS1779265626799http://carson.math.uwm.edu/jenkins/job/cam_BFB_coarse_res_gfortran_test/235/pdf_closure_module.F90af96de45b65cdea71bcbd4e2a22562062374a9221779217968000http://carson.math.uwm.edu/jenkins/user/hueblerGunther Hueblerhuebler@uwm.eduRemoving a couple variables from openacc statements to fix the clubb_driver_test - these were causing stats errors on the GPU for chi and chip2. This is technically BIT_CHANGING. 2026-05-19 14:12:48 -0500af96de45b65cdea71bcbd4e2a22562062374a922Removing a couple variables from openacc statements to fix the clubb_driver_test - these were causing stats errors on the GPU for chi and chip2. This is technically BIT_CHANGING.editpdf_closure_module.F90githttp://carson.math.uwm.edu/jenkins/user/hueblerGunther Hueblerfalse236http://carson.math.uwm.edu/jenkins/job/cam_BFB_coarse_res_gfortran_test/236/234http://carson.math.uwm.edu/jenkins/job/cam_BFB_coarse_res_gfortran_test/234/