Erik Kluzek

Hi @brianpm thanks for talking about this. First, I added it because I added the ability to read in the namelist on the CTSM side, and realized this means that...

In answer to your questions: > 1. we have use_fates .and. use_fates_sp, but am I correct that use_fates_sp cannot be .true. unless use_fates is? So we really need only have...

@adrifoster this looks to me like a little thing that we could just bring in like it is? Do you agree? It looks like a simple bit-for-bit refactoring to improve...

I'm pretty sure this comes from a phase we went through where we figured allocating everything even when not needed would speed up development. Because it does take time to...

Rethinking this. The performance impact of allocating arrays that are allocated, but never used is going to be minimal. Still good to lower total memory though. What does have an...

Yep #229 is the main thing. And #829 is a bigger refactoring that would also possibly be even better. Those are the two existing issues that I saw that made...

The CTSM error you get with this is: Successfully initialized run control settings ``` check_var: variable PCT_OCEAN is not on dataset Input surface dataset is: /glade/campaign/cesm/cesmdata/inputdata//lnd/clm2/surfdata_map/ctsm5.1.dev052/s urfdata_360x720cru_hist_78pfts_CMIP6_simyr1700_c210824.nc This surface dataset...

The problem is that it's reading in rpointer.rof (which is an empty file), rather than the rpointer.rof.2000-01-06-00000 with the datestamp in it.

One thing I see is that there is a call to io_rpfile that needs curDateTime sent into it. https://github.com/ESCOMP/mizuRoute/blob/cesm-coupling/route/build/src/write_simoutput_pio.f90#L427 that's all I see that needs to be done so far....

> [This line](https://github.com/ESCOMP/mizuRoute/blob/be8db1f8f70309ea1877645faaea43fbf8db1800/route/build/cpl/RtmMod.F90#L833) is rpointer file read. But, actually this has not been tested so far. so may need to look into why Yeah I think we should add more...