Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

run_neon not working as expected #1860

Closed
wwieder opened this issue Oct 1, 2022 · 4 comments
Closed

run_neon not working as expected #1860

wwieder opened this issue Oct 1, 2022 · 4 comments
Assignees
Labels
bug something is working incorrectly

Comments

@wwieder
Copy link
Contributor

wwieder commented Oct 1, 2022

Brief summary of bug

A few features of the NEON workflow still don't seem to be working as intended, these include

  1. Pointing to the wrong ndep and aerosols files
  2. Duplications in user_nl_* files that seem to come from cloning a base case and then copying additional information into the user_nl_* files.

The former is potentially a larger issues, while the later is just confusing (at least for me).

General bug information

Examples of both can be found in ad cases here:

/glade/scratch/wwieder/run_NEON_v2/tools/site_and_regional

Specifically BART.ad/user_nl_clm is pointing to

stream_fldfilename_ndep = '$DIN_LOC_ROOT/lnd/clm2/ndepdata/fndep_clm_f09_g17.CMIP6-SSP3-7.0-WACCM_2018-2030_monthly_c210826.nc'

with BART.ad/user_nl_datm_streams specifying

presaero.SSP3-7.0:datafiles = $DIN_LOC_ROOT/atm/cam/chem/trop_mozart_aero/aero/aerodep_clm_SSP370_b.e21.BWSSP370cmip6.f09_g17.CMIP6-SSP3-7.0-WACCM.001_2018-2030_monthly_0.9x1.25_c210826.nc
presaero.SSP3-7.0:year_first=2018

but BART.ad/CaseDocs/datm.streams.xml shows

<stream_info name="presndep.clim_2000">
   <datafiles>      <file>/glade/p/cesmdata/cseg/inputdata/lnd/clm2/ndepdata/fndep_clm_hist_b.e21.BWHIST.f09_g17.CMIP6-historical-WACCM.ensmean_1849-2015_monthly_0.9x1.25_c180926.nc</file>
   </datafiles>

Important details of your setup / configuration so we can reproduce the bug

This code is using ctsm5.1.dev110, with changes to cdeps to try and work around #1858

@wwieder wwieder added bug something is working incorrectly next this should get some attention in the next week or two. Normally each Thursday SE meeting. labels Oct 1, 2022
@wwieder wwieder added this to the NEON milestone Oct 1, 2022
@wwieder
Copy link
Contributor Author

wwieder commented Oct 1, 2022

@billsacks hopefully this is a simple fix we can bring in with the cdeps tag (once both are ready)?

@wwieder
Copy link
Contributor Author

wwieder commented Oct 1, 2022

I think this is a simple change to cime_config/usermods_dirs/NEON/defaults/shell_commands

./xmlchange DATM_PRESNDEP=SSP3-7.0
./xmlchange DATM_PRESO3=SSP3-7.0

@wwieder
Copy link
Contributor Author

wwieder commented Oct 1, 2022

as well as these additions to cime_config/usermods_dirs/NEON/defaults/user_nl_datm_streams

presndep.SSP3-7.0:datafiles = $DIN_LOC_ROOT/lnd/clm2/ndepdata/fndep_clm_f09_g17.CMIP6-SSP3-7.0-WACCM_2018-2030_monthly_c210826.nc
presndep.SSP3-7.0:year_first=2018
presndep.SSP3-7.0:year_last=2030
presndep.SSP3-7.0:year_align=2018
presndep.SSP3-7.0:dtlimit=30

preso3.SSP3-7.0:year_first=2018
preso3.SSP3-7.0:year_last=2030
preso3.SSP3-7.0:year_align=2018
preso3.SSP3-7.0:dtlimit=30

@wwieder
Copy link
Contributor Author

wwieder commented Oct 1, 2022

finally, for clarity should we remove the following from cime_config/usermods_dirs/NEON/defaults/user_nl_clm?

stream_fldfilename_ndep = '$DIN_LOC_ROOT/lnd/clm2/ndepdata/fndep_clm_f09_g17.CMIP6-SSP3-7.0-WACCM_2018-2030_monthly_c210826.nc'

glemieux added a commit to glemieux/ctsm that referenced this issue Oct 5, 2022
Fixes for NEON cases

Two sets of fixes for NEON cases:

(1) Fixes for leap year handling when there are discrepancies between
the model and the DATM forcing data in terms of whether the current year
is a leap year. This involved updates to the share and cdeps externals
from Jim Edwards.

(2) Other fixes for NEON cases from Will Wieder, as documented in
ESCOMP#1860
ekluzek added a commit that referenced this issue Oct 11, 2022
Fixes for NEON cases

Two sets of fixes for NEON cases:

(1) Fixes for leap year handling when there are discrepancies between
the model and the DATM forcing data in terms of whether the current year
is a leap year. This involved updates to the share and cdeps externals
from Jim Edwards.

(2) Other fixes for NEON cases from Will Wieder, as documented in
#1860

 Conflicts:
	Externals.cfg
	src/biogeophys/test/Photosynthesis_test/test_Photosynthesis.pf
	src/main/clm_initializeMod.F90
	src/main/clm_varpar.F90
	tools/README
	tools/mkmapdata/mkmapdata.sh
@billsacks billsacks removed the next this should get some attention in the next week or two. Normally each Thursday SE meeting. label Oct 20, 2022
@ekluzek ekluzek removed this from the NEON milestone Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something is working incorrectly
Projects
None yet
Development

No branches or pull requests

3 participants