forked from E3SM-Project/E3SM
-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enable HLM to create multiple columns for the vegetated landunit #2
Comments
Not that currently all calls to E3SM/components/elm/src/main/initGridCellsMod.F90 Lines 138 to 140 in 67abd00
|
glemieux
pushed a commit
that referenced
this issue
Oct 11, 2024
Addresses minor cleaning edits following second round of review
glemieux
pushed a commit
that referenced
this issue
Oct 11, 2024
glemieux
pushed a commit
that referenced
this issue
Oct 11, 2024
glemieux
pushed a commit
that referenced
this issue
Nov 21, 2024
Implement the MAMDryDep::run_impl() function.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The current model assumes a single column for the vegetated landunit, which is added upon gridcell initliazation:
E3SM/components/elm/src/main/initGridCellsMod.F90
Lines 347 to 348 in 67abd00
We will need to allow multiple soil columns for a vegetated land unit to be added during initialization such that when we have disturbance of the soil we move the appropriate patches and their area to the appropriate time-since-fire age class bin column. This movement will need to be triggered by a disturbance mechanism similar to what we do with fates.
This will require updating the column weighting for the relevant upstream hierarchical unit:
E3SM/components/elm/src/data_types/ColumnType.F90
Lines 34 to 40 in 67abd00
Tasks
The text was updated successfully, but these errors were encountered: