Skip to content
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

Update active crown fire per Scott and Reinhardt 2001, Rothermel 1977 #1

Open
wants to merge 1,091 commits into
base: active_crown_fire
Choose a base branch
from

Conversation

jkshuman
Copy link

Calculate required intensity to scorch crown and canopy, then re-calculate
rate of spread and fire intensity

Fixes: 573

User interface changes?: No

Code review:

Description:

Collaborators:

@adrifoster, @slevisconsulting

Expectation of Answer Changes:

Checklist:

  • My change requires a change to the documentation.
  • I have updated the in-code documentation .AND. (the technical note .OR. the wiki) accordingly.
  • 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: not tested

rgknox and others added 29 commits September 11, 2023 12:07
Small quality of life update to cohort age in create_cohort
This allows organ_param_id to be referenced in FatesCheckParams
when we are calculating the temperature of negative Rdark for each
pft.
…initfix

Fix error with large size initialisation in nocomp mode
In this tag, Jessica Needham adds a fix to nocomp initialization by adding in a call to height demography.
Changes all occurances of "hite" to "height"
@adrifoster adrifoster force-pushed the active_crown_Scott_2001 branch from c95ddab to 3090ea4 Compare March 12, 2024 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants