Fix error with large size initialisation in nocomp mode

[JessicaNeedham]

Fixes #1071. Initialisation by dbh was failing in no comp mode because height was not being initialised. This PR adds a call to h_allom so that trees can be demoted in canopy_structure if site spread has been updated in the first time step.

Collaborators:

Expectation of Answer Changes:

nocomop initialisation by dbh should now run, rather than result in an error.

Checklist:

• My change requires a change to the documentation.
• I have updated the in-code documentation .AND. (the technical note .OR. the wiki) accordingly.
• [ x] I have read the CONTRIBUTING document.
• FATES PASS/FAIL regression tests were run
• If answers were expected to change, evaluation was performed and provided

Test Results:

CTSM (or) E3SM (specify which) test hash-tag:

CTSM (or) E3SM (specify which) baseline hash-tag:

FATES baseline hash-tag:

Test Output:

Compiles and runs on Perlmutter with https://github.com/glemieux/E3SM/tree/lnd/fates-refactor E3SM branch and latest FATES commit.

[rgknox]

Looks good, makes sense.

Stretch goal: In the call to create_cohort, we've already updated that named argument to convey a zero aged cohort. How about that integer value of 1 that we pass. I think it is canopy layer. Perhaps we should use a named constant there too. Actually, I'm surprised we pass anything in, we shouldn't be defining the layer upon create cohort, that is an emergent phenomonena. It almost seems better to pass nothing and set it to uninitialized. I'll make an issue instead.

[rgknox]

Here is the new issue: #1082
So my suggestion of changing canopy layer to the create_patch call is out of scope and I see nothing else to do on this PR.

[rgknox]

running tests: /glade/scratch/rgknox/tests_0928-180603ch

tests pass, merging