Started by an SCM change273a05d60b6d7939a7f17776262bf8634a6f2702773a05d60b6d7939a7f17776262bf8634a6f2702773refs/remotes/origin/clubb_silhs_devela05d60b6d7939a7f17776262bf8634a6f2702773a05d60b6d7939a7f17776262bf8634a6f2702773refs/remotes/origin/clubb_silhs_devela05d60b6d7939a7f17776262bf8634a6f2702773a05d60b6d7939a7f17776262bf8634a6f2702773refs/remotes/origin/clubb_silhs_develhttps://github.com/larson-group/cam.gitfalse#2736265433580584cam_BFB_coarse_res_gfortran_test #273273false2738989FAILURE1790670427254http://carson.math.uwm.edu/jenkins/view/all/job/cam_BFB_coarse_res_gfortran_test/273/precipitation_fraction.F907c257a58ca47d40c2f20dd54cda1d085b11af2e11790609573000http://carson.math.uwm.edu/jenkins/user/noreplynoreplynoreply@github.comRemove upsilon 0 and 1 if branches (#1365) * Removed unnecessary upsilon_precip_frac_rat == 0 or 1 checks The read in parameter upsilon_precip_frac_rat is set in .in files. upsilon_precip_frac_rat describes the precipitation ratio between the first and second PDF component. The edge cases, all perticipitation in one PDF (upsilon=0 or 1) had their own if branches. These edge case branches are not strictly necessary as upsilon should never be set to 0 or 1 and we can live with a tiny precipitation fraction of precip_frac_tol in such a case. Furthermore a logic error was present which would never execute the if branch for the upsilon=0 case anyway. Per default upsilon is not set to 0 and afaik is not tested in the jenkins tests. However if upsilon is read in as 0 or 1 this commit would be technically BIT_CHANGING. A mirrored code removal in clubb-jax was also done. * changed comment about role of upsilon a bit 2026-09-28 10:32:53 -05007c257a58ca47d40c2f20dd54cda1d085b11af2e1Remove upsilon 0 and 1 if branches (#1365)editprecipitation_fraction.F90githttp://carson.math.uwm.edu/jenkins/user/noreplynoreplyfalse274http://carson.math.uwm.edu/jenkins/view/all/job/cam_BFB_coarse_res_gfortran_test/274/272http://carson.math.uwm.edu/jenkins/view/all/job/cam_BFB_coarse_res_gfortran_test/272/