Skip to content

Commit

Permalink
Merge pull request #916 from nusbaume/sparse_checkout_2_1
Browse files Browse the repository at this point in the history
Replace COSP Github-svn bridge with git sparse checkout for CESM2.1
  • Loading branch information
nusbaume authored Nov 27, 2023
2 parents d6d94f1 + 70623c8 commit 43488d4
Show file tree
Hide file tree
Showing 6 changed files with 103 additions and 36 deletions.
7 changes: 4 additions & 3 deletions Externals_CAM.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -21,9 +21,10 @@ required = True

[cosp2]
local_path = src/physics/cosp2/src
protocol = svn
repo_url = https://github.com/CFMIP/COSPv2.0/tags/
tag = v2.1.4cesm/src
protocol = git
repo_url = https://github.com/CFMIP/COSPv2.0
sparse = ../.cosp_sparse_checkout
tag = v2.1.4cesm
required = True

[externals_description]
Expand Down
22 changes: 11 additions & 11 deletions bld/configure
Original file line number Diff line number Diff line change
Expand Up @@ -121,9 +121,9 @@ OPTIONS
test_tracers, test_tracers2].
Default: none.
-chem <name> Build CAM with specified prognostic chemistry package
[ trop_mam3 | trop_mam4 | trop_mam7 | trop_mozart | trop_strat_mam4_vbs | trop_strat_mam4_vbsext |
waccm_ma | waccm_mad | waccm_mad_mam4 | waccm_ma_mam4 |
waccm_ma_sulfur | waccm_sc | waccm_sc_mam4 | waccm_tsmlt_mam4 |
[ trop_mam3 | trop_mam4 | trop_mam7 | trop_mozart | trop_strat_mam4_vbs | trop_strat_mam4_vbsext |
waccm_ma | waccm_mad | waccm_mad_mam4 | waccm_ma_mam4 |
waccm_ma_sulfur | waccm_sc | waccm_sc_mam4 | waccm_tsmlt_mam4 |
terminator | none ].
Default: trop_mam4 for cam6 and trop_mam3 for cam5.
-[no]clubb_sgs Switch on [off] CLUBB_SGS. Default: on for cam6, otherwise off.
Expand Down Expand Up @@ -1929,7 +1929,7 @@ if ($cfg_ref->get('analytic_ic')) {

#WACCM-X extended thermosphere/ionosphere model
if ($waccmx) {
$cfg_cppdefs .= ' -DWACCMX_PHYS';
$cfg_cppdefs .= ' -DWACCMX_PHYS';
if (($dyn_pkg ne 'fv') and ($ionos ne 'none')) {
die "ERROR: Ionosphere is only available for FV dycore \n";
}
Expand Down Expand Up @@ -3075,13 +3075,13 @@ sub write_cosp_makefile
CAM_BLD := $cam_bld
COSP_PATH := $cam_dir/src/physics/cosp2
ISCCP_PATH := $cam_dir/src/physics/cosp2/src/simulator/icarus
RS_PATH := $cam_dir/src/physics/cosp2/src/simulator/quickbeam
RT_PATH := $cam_dir/src/physics/cosp2/src/simulator/rttov
CS_PATH := $cam_dir/src/physics/cosp2/src/simulator/actsim
MISR_PATH := $cam_dir/src/physics/cosp2/src/simulator/MISR_simulator
MODIS_PATH := $cam_dir/src/physics/cosp2/src/simulator/MODIS_simulator
PARASOL_PATH := $cam_dir/src/physics/cosp2/src/simulator/parasol
ISCCP_PATH := $cam_dir/src/physics/cosp2/src/src/simulator/icarus
RS_PATH := $cam_dir/src/physics/cosp2/src/src/simulator/quickbeam
RT_PATH := $cam_dir/src/physics/cosp2/src/src/simulator/rttov
CS_PATH := $cam_dir/src/physics/cosp2/src/src/simulator/actsim
MISR_PATH := $cam_dir/src/physics/cosp2/src/src/simulator/MISR_simulator
MODIS_PATH := $cam_dir/src/physics/cosp2/src/src/simulator/MODIS_simulator
PARASOL_PATH := $cam_dir/src/physics/cosp2/src/src/simulator/parasol
EOF

Expand Down
67 changes: 66 additions & 1 deletion doc/ChangeLog
Original file line number Diff line number Diff line change
@@ -1,8 +1,73 @@
===============================================================

Tag name: cam_cesm2_1_rel_59
Originator(s): nusbaume
Date: 27 Nov 2023
One-line Summary: removes svn sparse checkout
Github PR URL: https://github.com/ESCOMP/CAM/pull/916

Purpose of changes (include the issue number and title text for each relevant GitHub issue):

Use git sparse checkout for all repos, combine clubb and silhs into clubb.
git sparse checkout has been available in manage_externals since version 1.2.1.

Fixes #912 -> cosp2 svn access to github support is ending soon

Describe any changes made to build system:

Build path were modified in the "configure" Perl script, as well
as in the COSP Makefile.in file, in order to account for the
new source code paths.

Describe any changes made to the namelist: N/A

List any changes to the defaults for the boundary datasets: N/A

Describe any substantial timing or memory changes: N/A

Code reviewed by: cacraigucar

List all files eliminated: N/A

List all files added and what they do:

A src/physics/cosp2/.cosp_sparse_checkout
- Provides information on how to do the git sparse checkout of COSP

List all existing files that have been modified, and describe the changes:

M Externals_CAM.cfg
- Change the Github-SVN bridge method to git sparse checkout

M bld/configure
- Update source code paths for COSP

M src/physics/cosp2/Makefile.in
- Update source code paths for relevant COSP files.

M test/system/test_driver.sh
- Add qsub flag needed for old test system to run on Cheyenne.

If there were any failures reported from running test_driver.sh on any test
platform, and checkin with these failures has been OK'd by the gatekeeper,
then copy the lines from the td.*.status files for the failed tests to the
appropriate machine below. All failed tests must be justified.

cheyenne/intel: bit-for-bit

cheyenne/intel/aux_cam: bit-for-bit

izumi/nag: bit-for-bit

izumi/pgi: SKIPPED - NO LONGER SUPPORTED

Summarize any changes to answers: ALL SUPPORTED TESTS PASS

===============================================================

Tag name: cam_cesm2_1_rel_r58
Originator(s): fischer
Date: 30 May 2023
Date: 30 May 2023
One-line Summary: Add use case for BSSP585smbb
Github PR URL: https://github.com/ESCOMP/CAM/pull/822

Expand Down
1 change: 1 addition & 0 deletions src/physics/cosp2/.cosp_sparse_checkout
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
/src/
36 changes: 18 additions & 18 deletions src/physics/cosp2/Makefile.in
Original file line number Diff line number Diff line change
Expand Up @@ -65,10 +65,10 @@ cosp_grLidar532_interface.o: cosp_kinds.o
cosp_atlid_interface.o : cosp_kinds.o
cosp_cloudsat_interface.o : cosp_kinds.o cosp_config.o quickbeam.o
cosp_isccp_interface.o : cosp_kinds.o icarus.o
cosp_misr_interface.o : cosp_kinds.o
cosp_misr_interface.o : cosp_kinds.o
cosp_modis_interface.o : cosp_kinds.o cosp_config.o modis_simulator.o
cosp_rttov_interfaceSTUB.o : cosp_kinds.o cosp_config.o cosp_rttovSTUB.o
cosp_parasol_interface.o : cosp_kinds.o
cosp_parasol_interface.o : cosp_kinds.o
cosp_rttovSTUB.o : cosp_kinds.o cosp_config.o cosp_constants.o
MISR_simulator.o : cosp_kinds.o cosp_config.o cosp_stats.o
modis_simulator.o : cosp_kinds.o cosp_config.o cosp_stats.o
Expand All @@ -82,7 +82,7 @@ scops.o : cosp_kinds.o mo_rng.o cosp_errorHandling.o
prec_scops.o : cosp_kinds.o cosp_config.o
cosp_optics.o : cosp_kinds.o cosp_constants.o modis_simulator.o
quickbeam_optics.o : cosp_kinds.o cosp_config.o cosp_constants.o quickbeam.o \
cosp_errorHandling.o array_lib.o math_lib.o optics_lib.o
cosp_errorHandling.o array_lib.o math_lib.o optics_lib.o
optics_lib.o : cosp_kinds.o cosp_errorHandling.o
array_lib.o : cosp_kinds.o cosp_errorHandling.o
math_lib.o : cosp_kinds.o array_lib.o mrgrnk.o
Expand All @@ -107,37 +107,37 @@ quickbeam.o: $(RS_PATH)/quickbeam.F90
MISR_simulator.o : $(MISR_PATH)/MISR_simulator.F90
$(F90) $(F90FLAGS) -c $<

modis_simulator.o : $(MODIS_PATH)/modis_simulator.F90
modis_simulator.o : $(MODIS_PATH)/modis_simulator.F90
$(F90) $(F90FLAGS) -c $<

cosp_rttov_interfaceSTUB.o : $(COSP_PATH)/src/simulator/cosp_rttov_interfaceSTUB.F90
cosp_rttov_interfaceSTUB.o : $(COSP_PATH)/src/src/simulator/cosp_rttov_interfaceSTUB.F90
$(F90) $(F90FLAGS) -c $<

cosp_misr_interface.o : $(COSP_PATH)/src/simulator/cosp_misr_interface.F90
cosp_misr_interface.o : $(COSP_PATH)/src/src/simulator/cosp_misr_interface.F90
$(F90) $(F90FLAGS) -c $<

cosp_modis_interface.o : $(COSP_PATH)/src/simulator/cosp_modis_interface.F90
cosp_modis_interface.o : $(COSP_PATH)/src/src/simulator/cosp_modis_interface.F90
$(F90) $(F90FLAGS) -c $<

cosp_isccp_interface.o : $(COSP_PATH)/src/simulator/cosp_isccp_interface.F90
cosp_isccp_interface.o : $(COSP_PATH)/src/src/simulator/cosp_isccp_interface.F90
$(F90) $(F90FLAGS) -c $<

cosp_calipso_interface.o : $(COSP_PATH)/src/simulator/cosp_calipso_interface.F90
cosp_calipso_interface.o : $(COSP_PATH)/src/src/simulator/cosp_calipso_interface.F90
$(F90) $(F90FLAGS) -c $<

cosp_grLidar532_interface.o : $(COSP_PATH)/src/simulator/cosp_grLidar532_interface.F90
cosp_grLidar532_interface.o : $(COSP_PATH)/src/src/simulator/cosp_grLidar532_interface.F90
$(F90) $(F90FLAGS) -c $<

cosp_atlid_interface.o : $(COSP_PATH)/src/simulator/cosp_atlid_interface.F90
cosp_atlid_interface.o : $(COSP_PATH)/src/src/simulator/cosp_atlid_interface.F90
$(F90) $(F90FLAGS) -c $<

cosp_cloudsat_interface.o : $(COSP_PATH)/src/simulator/cosp_cloudsat_interface.F90
cosp_cloudsat_interface.o : $(COSP_PATH)/src/src/simulator/cosp_cloudsat_interface.F90
$(F90) $(F90FLAGS) -c $<

cosp_parasol_interface.o : $(COSP_PATH)/src/simulator/cosp_parasol_interface.F90
cosp_parasol_interface.o : $(COSP_PATH)/src/src/simulator/cosp_parasol_interface.F90
$(F90) $(F90FLAGS) -c $<

cosp_rttovSTUB.o : $(RT_PATH)/cosp_rttovSTUB.F90
cosp_rttovSTUB.o : $(RT_PATH)/cosp_rttovSTUB.F90
$(F90) $(F90FLAGS) -c $<

lidar_simulator.o : $(CS_PATH)/lidar_simulator.F90
Expand All @@ -146,19 +146,19 @@ lidar_simulator.o : $(CS_PATH)/lidar_simulator.F90
parasol.o : $(PARASOL_PATH)/parasol.F90
$(F90) $(F90FLAGS) -c $<

cosp_constants.o : $(COSP_PATH)/src/cosp_constants.F90
cosp_constants.o : $(COSP_PATH)/src/src/cosp_constants.F90
$(F90) $(F90FLAGS) -c $<

cosp_kinds.o : $(COSP_PATH)/cosp_kinds.F90
$(F90) $(F90FLAGS) -c $<

cosp_config.o : $(COSP_PATH)/src/cosp_config.F90
cosp_config.o : $(COSP_PATH)/src/src/cosp_config.F90
$(F90) $(F90FLAGS) -c $<

cosp.o : $(COSP_PATH)/src/cosp.F90
cosp.o : $(COSP_PATH)/src/src/cosp.F90
$(F90) $(F90FLAGS) -c $<

cosp_stats.o : $(COSP_PATH)/src/cosp_stats.F90
cosp_stats.o : $(COSP_PATH)/src/src/cosp_stats.F90
$(F90) $(F90FLAGS) -c $<

# COSPv1.4 interface
Expand Down
6 changes: 3 additions & 3 deletions test/system/test_driver.sh
Original file line number Diff line number Diff line change
Expand Up @@ -937,7 +937,7 @@ if [ -n "${submit_script_cb}" ]; then
case $hostname in
# cheyenne
chey* | r* )
batch_queue_submit='qsub '
batch_queue_submit='qsub -V'
;;
*)
echo "ERROR: machine $hostname not currently supported for batch builds"
Expand Down Expand Up @@ -1082,7 +1082,7 @@ if [ "${cesm_test_suite}" != "none" -a -n "${cesm_test_mach}" ]; then
echo "cd ${script_dir}" >> ${submit_script_cime}
echo './create_test' ${testargs} >> ${submit_script_cime}
chmod u+x ${submit_script_cime}
qsub ${submit_script_cime}
qsub -V ${submit_script_cime}
fi

if [ "${hostname:0:6}" == "hobart" ]; then
Expand Down Expand Up @@ -1116,7 +1116,7 @@ if $run_cam_regression; then
case $hostname in
##cheyenne
ch* | r* )
qsub ${submit_script_cb}
qsub -V ${submit_script_cb}
;;

##hobart
Expand Down

0 comments on commit 43488d4

Please sign in to comment.