Results 80 comments of Ryan Knox

I ran three tests, using Levels "0,0" "1,1" and "2,2". Each test used the f45 grid, fixed biogeography and nocomp, and benefited from 15 years of spinup prior to the...

Thanks for pointing this out @olyson . My sense is that this is pointing out a vulnerability in the procedure: psi_from_th(). Initializing it with non frozen soils should work for...

It is odd though, if its a div0, then the saturation water content is the zero, which should be a soil parameter.

Noting that this is where we setup the saturated water content in a cold-start: https://github.com/NGEET/fates/blob/sci.1.72.0_api.33.0.0/biogeophys/FatesPlantHydraulicsMod.F90#L1575-L1581

yes, appears to be the same error as well: > ERROR: module command /glade/u/apps/dav/opt/lmod/7.7.29/libexec/lmod python purge failed with message: /glade/u/apps/dav/opt/lua/5.3.4/bin/lua: error while loading shared libraries: libreadline.so.6: cannot open shared object...

I'm looking through this more closely, but I think that the change to "dr" should propogate into FATES results as well, which should explain the diffs. https://github.com/ESCOMP/CTSM/pull/2348/files#diff-a313a6093d335d65fdb56abaedbbc3416f8b8f5f095f828ddd703f907d57526bR1659

FATES without reduced complexity (fixed biogeography) should not use any of the patch level data in the surface file, so it shouldn't matter how the PFTs are setup, or if...

@billsacks, we test fates with 1 test using an f09_g16 grid, which seems to be very expensive according to your chart (which makes sense being so large..). It does not...

>(1) Whether the Clm45Fates f09_g17 test can be cut out (does anyone run Fates with Clm45? We still do have coarser-resolution Clm45Fates tests.) I don't think so... we could at...