Skip to content

Commit

Permalink
Merge tag 'ctsm1.0.dev084' into rel2master
Browse files Browse the repository at this point in the history
Bounds assertion for C isotopes when threading is on

Previously, the bounds of c13 and c14 in subroutines TruncateCStates and
TruncateCandNStates were not checked when threading was on. This PR allows us
to handle c13 and c14 arrays like everything else in CTSM including (bounds
checking).

This PR resolves issue ESCOMP#811
  • Loading branch information
ekluzek committed Feb 25, 2020
2 parents d4c0a0b + 95fc9e1 commit de37884
Show file tree
Hide file tree
Showing 28 changed files with 1,215 additions and 480 deletions.
36 changes: 0 additions & 36 deletions .github/ISSUE_TEMPLATE/02_support_needed.md

This file was deleted.

File renamed without changes.
5 changes: 5 additions & 0 deletions .github/ISSUE_TEMPLATE/config.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
blank_issues_enabled: true
contact_links:
- name: CESM forums
url: https://xenforo.cgd.ucar.edu/cesm/
about: For support with model use, troubleshooting, etc., please use the CTSM forum
2 changes: 1 addition & 1 deletion Externals.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ required = True
local_path = cime
protocol = git
repo_url = https://github.com/ESMCI/cime
tag = cime5.8.15
tag = branch_tags/cime5.8.15_a01
externals = ../Externals_cime.cfg
required = True

Expand Down
5 changes: 5 additions & 0 deletions README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,11 @@ and

https://github.com/ESCOMP/ctsm/wiki/Recommended-git-setup

For support with model use, troubleshooting, etc., please use the CTSM forum (or other
appropriate forum) here:

https://xenforo.cgd.ucar.edu/cesm/

To get updates on CTSM tags and important notes on CTSM developments
join our low traffic email list:

Expand Down
2 changes: 1 addition & 1 deletion bld/namelist_files/namelist_defaults_ctsm.xml
Original file line number Diff line number Diff line change
Expand Up @@ -1025,7 +1025,7 @@ lnd/clm2/surfdata_map/landuse.timeseries_ne30np4_hist_16pfts_Irrig_CMIP6_simyr18
<use_fun phys="clm5_0" use_cn=".true." use_nitrif_denitrif=".true.">.true.</use_fun>
<use_fun >.false.</use_fun>

<br_root phys="clm5_0" use_cn=".true." use_fun=".true.">0.83d-06</br_root>
<br_root>0.83d-06</br_root>

<!-- Scalar of leaf respiration to vcmax (used for SP mode and with luna)-->
<leaf_mr_vcm phys="clm5_0" >0.015d00</leaf_mr_vcm>
Expand Down
26 changes: 13 additions & 13 deletions cime_config/testdefs/testlist_clm.xml
Original file line number Diff line number Diff line change
Expand Up @@ -254,7 +254,7 @@
<option name="wallclock">00:20:00</option>
</options>
</test>
<test name="SMS_P720x1_Ln3" grid="hcru_hcru" compset="I2000Clm50BgcCruGs" testmods="clm/coldStart">
<test name="SMS_P720x1_Ln6" grid="hcru_hcru" compset="I2000Clm50BgcCruGs" testmods="clm/coldStart">
<machines>
<machine name="cheyenne" compiler="intel" category="aux_clm"/>
</machines>
Expand Down Expand Up @@ -431,7 +431,7 @@
<machine name="cheyenne" compiler="intel" category="prebeta"/>
</machines>
<options>
<option name="wallclock">0:20:00</option>
<option name="wallclock">00:20:00</option>
<option name="comment" >Do a test with RTM and flooding on as that also impacts CLM code</option>
</options>
</test>
Expand Down Expand Up @@ -759,7 +759,7 @@
<machines>
<machine name="cheyenne" compiler="intel" category="aux_clm">
<options>
<option name="wallclock">0:30:00</option>
<option name="wallclock">00:30:00</option>
<option name="comment">Want an ERP test covering some non-default irrigation options. Long enough so that we're likely to exercise the various groundwater irrigation code.</option>
</options>
</machine>
Expand Down Expand Up @@ -1699,7 +1699,7 @@
<machines>
<machine name="izumi" compiler="nag" category="aux_clm">
<options>
<option name="wallclock">0:20:00</option>
<option name="wallclock">00:20:00</option>
<option name="comment">Debug test covering some non-default irrigation options.</option>
</options>
</machine>
Expand All @@ -1710,7 +1710,7 @@

<machine name="izumi" compiler="intel" category="aux_clm">
<options>
<option name="wallclock">0:30:00</option>
<option name="wallclock">00:30:00</option>
<option name="comment">Include a tracer consistency check in debug mode.</option>
</options>
</machine>
Expand All @@ -1722,7 +1722,7 @@

<machine name="cheyenne" compiler="intel" category="aux_clm">
<options>
<option name="wallclock">0:30:00</option>
<option name="wallclock">00:30:00</option>
<option name="comment">A debug ERP test of the NWP configuration with active BGC and CROP.</option>
</options>
</machine>
Expand All @@ -1733,7 +1733,7 @@
<machines>
<machine name="cheyenne" compiler="gnu" category="aux_clm">
<options>
<option name="wallclock">0:30:00</option>
<option name="wallclock">00:30:00</option>
<option name="comment">Ensure that turning on water tracers doesn't change answers. Cold start for now, until we can use initial conditions from a non-isotope case in an isotope case; once we can do that, this should be changed to not be cold start (e.g., 5-day decStart transient test: see also https://github.com/ESCOMP/ctsm/issues/495#issuecomment-516619853).</option>
</options>
</machine>
Expand All @@ -1744,7 +1744,7 @@

<machine name="cheyenne" compiler="intel" category="aux_clm">
<options>
<option name="wallclock">0:30:00</option>
<option name="wallclock">00:30:00</option>
<option name="comment">Include a debug ERP test of the NWP configuration.</option>
</options>
</machine>
Expand All @@ -1756,7 +1756,7 @@

<machine name="cheyenne" compiler="gnu" category="aux_clm">
<options>
<option name="wallclock">0:30:00</option>
<option name="wallclock">00:30:00</option>
<option name="comment">Include a short smoke test covering the nldas2 grid and the I2000Ctsm50NwpSpNldasGs compset, which uses NLDAS datm forcing.</option>
</options>
</machine>
Expand All @@ -1768,7 +1768,7 @@

<machine name="cheyenne" compiler="gnu" category="aux_clm">
<options>
<option name="wallclock">0:30:00</option>
<option name="wallclock">00:30:00</option>
<option name="comment">Include a short smoke test covering the nldas2 grid and the I2000Ctsm50NwpSpNldasRsGs compset, which uses NLDAS datm forcing.</option>
</options>
</machine>
Expand Down Expand Up @@ -1963,7 +1963,7 @@
<machines>
<machine name="cheyenne" compiler="intel" category="aux_clm">
<options>
<option name="wallclock">0:30</option>
<option name="wallclock">00:30:00</option>
<option name="comment">Include a test of the NUOPC cap</option>
</options>
</machine>
Expand All @@ -1973,7 +1973,7 @@
<machines>
<machine name="cheyenne" compiler="intel" category="aux_clm">
<options>
<option name="wallclock">0:30:00</option>
<option name="wallclock">00:30:00</option>
<option name="comment">Can use this test to determine if there are significant throughput changes, at least for this common and important configuration. Note that this deliberately doesn't have any testmods in order to (1) avoid doing history output (because the timing of output can be very variable, and mixing output timing with other aspects of model time can be confusing), and (2) generally keep the test replicating a production configuration as closely as possible (so, for example, we do NOT set BFBFLAG=TRUE for this test).</option>
</options>
</machine>
Expand All @@ -1983,7 +1983,7 @@
<machines>
<machine name="cheyenne" compiler="intel" category="aux_clm">
<options>
<option name="wallclock">0:30:00</option>
<option name="wallclock">00:30:00</option>
<option name="comment">This test runs CTSM's Fortran unit tests. We're abusing the system test infrastructure to run these, so that a run of the test suite results in the unit tests being run as well. Grid and compset are irrelevant here, except that compset must be one that includes CTSM in order for CIME to find the test definition.</option>
</options>
</machine>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ hist_dov2xy(1) = .true.
hist_nhtfrq(1) = 0
hist_type1d_pertape(1) = ' '
hist_fexcl1 += 'PCT_GLC_MEC','PCT_NAT_PFT'
hist_fincl1 += 'EFLX_LH_TOT_ICE', 'FIRE_ICE', 'FLDS_ICE', 'FSH_ICE', 'FSNO_ICE', 'FSR_ICE', 'QFLX_SUB_SNOW_ICE', 'QRUNOFF_ICE', 'QSNOFRZ_ICE', 'QSNOMELT_ICE', 'RAIN_ICE', 'SNOW_ICE', 'SNOWICE_ICE', 'SNOWLIQ_ICE', 'SNOTXMASS_ICE', 'TG_ICE', 'TOPO_COL_ICE', 'TSA_ICE', 'TSOI_ICE'
hist_fincl1 += 'EFLX_LH_TOT_ICE', 'FIRE_ICE', 'FLDS_ICE', 'FSH_ICE', 'FSNO_ICE', 'FSR_ICE', 'QFLX_SOLIDEVAP_FROM_TOP_LAYER_ICE', 'QRUNOFF_ICE', 'QSNOFRZ_ICE', 'QSNOMELT_ICE', 'RAIN_ICE', 'SNOW_ICE', 'SNOWICE_ICE', 'SNOWLIQ_ICE', 'SNOTXMASS_ICE', 'TG_ICE', 'TOPO_COL_ICE', 'TSA_ICE', 'TSOI_ICE'

! h1 stream (monthly average, finest sub-grid)
! Emon, Lmon
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ hist_mfilt(6) = 365
hist_dov2xy(6) = .true.
hist_nhtfrq(6) = -24
hist_type1d_pertape(6) = ' '
hist_fincl6 += 'SOILWATER_10CM', 'TOTSOILLIQ', 'TOTSOILICE', 'EFLX_LH_TOT', 'FSH', 'FGR12', 'FSM', 'QSNOEVAP', 'TLAI', 'QDRAI', 'QDRAI_PERCH', 'QOVER', 'QFLX_SUB_SNOW', 'FSA', 'FIRA', 'H2OSNO', 'SNOCAN', 'QSNOFRZ', 'QFLX_SNOW_DRAIN', 'SNOWDP', 'H2OSFC', 'TV', 'TG', 'TAUX', 'TAUY', 'QVEGT', 'TWS', 'H2OCAN', 'QVEGE', 'QSOIL', 'TSKIN', 'FSDS','FSNO','SNOFSRVD','SNOFSRVI','SNOFSRND','SNOFSRNI','FSDSVD','FSDSVI','FSDSND','FSDSNI','SNOWLIQ','SOILICE','SOILLIQ','QINTR','SNOBCMSL','TSOI','SNOTXMASS','SNOWICE','SNOWLIQ','QRUNOFF','RAIN','SNOW'
hist_fincl6 += 'SOILWATER_10CM', 'TOTSOILLIQ', 'TOTSOILICE', 'EFLX_LH_TOT', 'FSH', 'FGR12', 'FSM', 'QSNOEVAP', 'TLAI', 'QDRAI', 'QDRAI_PERCH', 'QOVER', 'QFLX_SOLIDEVAP_FROM_TOP_LAYER', 'FSA', 'FIRA', 'H2OSNO', 'SNOCAN', 'QSNOFRZ', 'QFLX_SNOW_DRAIN', 'SNOWDP', 'H2OSFC', 'TV', 'TG', 'TAUX', 'TAUY', 'QVEGT', 'TWS', 'H2OCAN', 'QVEGE', 'QSOIL', 'TSKIN', 'FSDS','FSNO','SNOFSRVD','SNOFSRVI','SNOFSRND','SNOFSRNI','FSDSVD','FSDSVI','FSDSND','FSDSNI','SNOWLIQ','SOILICE','SOILLIQ','QINTR','SNOBCMSL','TSOI','SNOTXMASS','SNOWICE','SNOWLIQ','QRUNOFF','RAIN','SNOW'

! h6 stream (daily average, landunit-level)
! Eday
Expand All @@ -21,10 +21,10 @@ hist_fincl7 += 'TREFMXAV','TREFMNAV'
! h7 stream (3-hourly average, gridcell-level)
! 3hr, E3hr, CF3hr
! 3hr requires QRUNOFF for time mean, and SOILWATER_10CM, TSKIN for time point (I)
! CF3hr requires QFLX_SUB_SNOW for time point (I)
! CF3hr requires QFLX_SOLIDEVAP_FROM_TOP_LAYER for time point (I)
hist_mfilt(8) = 2920
hist_dov2xy(8) = .true.
hist_nhtfrq(8) = -3
hist_type1d_pertape(8) = ' '
hist_fincl8 += 'TSA','RH2M','SOILWATER_10CM:I','FSH','EFLX_LH_TOT','FSDS','QRUNOFF','QFLX_SUB_SNOW:I','TSKIN:I'
hist_fincl8 += 'TSA','RH2M','SOILWATER_10CM:I','FSH','EFLX_LH_TOT','FSDS','QRUNOFF','QFLX_SOLIDEVAP_FROM_TOP_LAYER:I','TSKIN:I'

Loading

0 comments on commit de37884

Please sign in to comment.