Jim Edwards

@ekluzek I'm working on fixing the limit issues, I don't think you should back out your change.

After careful examination and experiments with the test SMS_Ly1_Mmpi-serial_Vnuopc.1x1_vancouverCAN.I1PtClm50SpRs.cheyenne_gnu.clm-output_sp_highfreq I have determined that the limit code in dshr_stream_mod.F90 is working correctly as written. Out of the box the test results...

@ekluzek I also tried the case mentioned in https://github.com/ESCOMP/CTSM/issues/1295 In this case with ``` model_year_align_ndep = 2018 ndep_taxmode = 'limit' ``` it fails with: ``` ERROR: (shr_stream_findBounds) ERROR: rDateIn lt...

I think that the correct settings for this case are ``` model_year_align_ndep = 1850 ndep_taxmode = 'limit' ndep_varlist = 'NDEP_month' ndepmapalgo = 'bilinear' stream_fldfilename_ndep = '/glade/p/cesmdata/cseg/inputdata/lnd/clm2/ndepdata/fndep_clm_f09_g17.CMIP6-SSP3-7.0-WACCM_1849-2101_monthly_c191007.nc' stream_year_first_ndep = 1850 stream_year_last_ndep...

I'm really confused about that - it would imply that you want your 2018 run to use the 1850 data in the file when the file does contain 2018 data...

The datm.streams.xml file now has both pairs in it, however there is no provision in cdeps to rotate more than one vector pair in a stream. @mvertens

@mvertens do I understand correctly that this will also require a change in mosart or did I get that wrong?

The proposed fix for this is to move nuopc_shr_methods.F90 into [cdeps/share](https://github.com/ESCOMP/CDEPS/pull/269) and [cesm_share/src](https://github.com/ESCOMP/CESM_share/pull/47)

@DeniseWorthen One thing that I wasn't sure of is if you use the same *_cpl_dt variables as cesm does. If not we will need to figure out how to do...

@DeniseWorthen I've added more error checking so that if someone in UFS does try to use the nsteps option it will throw an error. I also tested REST_OPTION='end' and fixed...