Keith Oleson

I've updated the initial PR comment.

I've reviewed this. The only difference I see is that I believe this line in the ctsm5.2 mksurfdata branch: write (6,*) subname, 'Error: sum of special land units nearly 100%...

Sam is fixing the item I identified on the branch. So I am closing this.

Thanks Bill, this was very useful information. We are meeting bi-weekly on this project and will contact you when we need help, thanks!

@billsacks , I'm a bit confused by the PCT_CROP_MAX that you mentioned and similar variables that are intended to provide a reasonable mask field. We were going to do something...

Performance notes for 5 year I2000 cases: "allurban": 152 yrs/day, 289 pe-hr/simulated_year normal: 172 yrs/day, 254 pe-hr/simulated_year So, about 12% slower when running 3 urban landunits in every gridcell.

I spoke to Bill about this. Basically, we should proceed with implementing the raw dynamic urban data into mksurfdata. As long as we use a recent version of ctsm master,...

@billsacks, we are wondering if it makes sense to issue a PR on our work thus far; task 4 (updating model code to read in and execute a landuse timeseries...

Regarding implementing transient urban into the mksurfdata routines...the current method replaces bare soil with urban. If there is not enough bare soil, then both PCT_NATVEG and PCT_CROP are reduced proportionally...

Thanks @lawrencepj1 , yes we will have PCT_URBAN on the transient landuse.timeseries file. I see the following code in dynLandunitAreaMod.F90 that controls the priority of which landunits get decreased if...