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

Feature/adarmenov/ignio #699

Merged
merged 40 commits into from
Feb 27, 2023
Merged

Feature/adarmenov/ignio #699

merged 40 commits into from
Feb 27, 2023

Conversation

adarmenov
Copy link
Contributor

IGNI is a new component that integrates fire weather observations into fuel moisture codes and fire behavior indexes based on the CFFWI system.

sdrabenh and others added 30 commits September 12, 2022 16:31
Wind at 2m, temperature and humidity at 10m,
are not used, hence they can be removed from the
import spec.
This refactoring reproduces the initial implementation.
There is no notable speed up. Slightly smaller code,
however some of the changes may have resulted in more
difficult to read code.
Define the CFFWI functions as elemental.
Tidy up comments.
…al use

Monthly day length adjustment factors for Drought Code (Lf)
can be applied for northern and southern hemispheres. Based on
Appendix 3 in Lawson and Armitage (2008).

Add utility test for the monthly day length adjustment factors
for Drought Code (Lf).
Hourly FFMC is based on:
  - A method of computing fine fuel moisture content
    throughout the diurnal cycle, C.E. Van Wagner, 1977.
  - A comparison of hourly fine fuel moisture code calculations
    within Canada, K. Anderson, 2009.
For now the FWI indexes are calculated daily.
The FF scale in the reference documents is:
  m = 147.2*(101 - F) / (59.5 + F)
  F = 59.5*(250 - m)/(147.2 + m)

A. Cantin et al., however recommend a more precise version,
especially for hourly or finer timesteps:
  m = 147.27723*(101 - F) / (59.5 + F)
  F = 59.5*(250 - m)/(147.27723 + m)
The wind term used in ISI is modified to follow the
FBP system - see Section 7.2.2, Eq. 53 and Eq. 53a
in 'Development and Structure of the Canadian Forest
Fire Behavior Prediction System, Technical ReportST-X-3,
Forestry Canada, 1992.'
Grass fuel moisture code (GFMC) is based on:
    Wotton, B.M., A grass moisture model for the Canadian
    Forest Fire Danger Rating System, 2009.
The CFFWI implementation in GEOS now provides
both daily and hourly fuel moisture codes and
fire behavior indexes.
Hourly and daily FFMC, DMC, DC, ISI, FWI and DSR
are now included in the SURFACE export spec.
For now GFMC is computed with fixed nominal fine fuel load.
The nominal fine fuel load was updated to 0.35 kg m-2 based on
the recommended value in Section 3.1.2 in 'Updates and
Revisions to the 1992 Canadian Forest Fire Behavior
Prediction System', GLC-X-10, 2009.
Default to LST local noon method until MAPL_SunGetLocalSolarHourAngle()
makes it into the main MAPL branch.
@adarmenov adarmenov added enhancement New feature or request 0 diff The changes in this pull request have verified to be zero-diff with the target branch. labels Feb 1, 2023
@adarmenov adarmenov requested review from a team as code owners February 1, 2023 18:41
@adarmenov adarmenov self-assigned this Feb 1, 2023
Copy link
Contributor

@sanAkel sanAkel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewing on behalf of @GEOS-ESM/ocean-team

@gmao-rreichle
Copy link
Contributor

Based on a brief inspection of the changes this looks ok to me from the land perspective.
To be safe, @biljanaorescanin should run the standard GEOSldas tests before the @GEOS-ESM/land-team approves.
(Also needs updating to the latest develop branch.)

Copy link
Contributor

@zhaobin74 zhaobin74 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not related to landice. Approve.

@adarmenov
Copy link
Contributor Author

... (Also needs updating to the latest develop branch.)

@gmao-rreichle Addressed with 04821aa

@adarmenov adarmenov requested a review from sanAkel February 1, 2023 19:36
Copy link
Collaborator

@tclune tclune left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please note that CMake provides a mechanism to register tests, and thereby make it easier for us to incorporate them into CI testing. It would also avoid hardcoding the compiler (gfortran in this case)

If at all interested please contact @mathomp4 or myself.

@sanAkel
Copy link
Contributor

sanAkel commented Feb 1, 2023

Please note that CMake provides a mechanism to register tests, and thereby make it easier for us to incorporate them into CI testing. It would also avoid hardcoding the compiler (gfortran in this case)

If at all interested please contact @mathomp4 or myself.

Yes, I'm interested and will reach out to @mathomp4. Thanks!

@adarmenov adarmenov removed their assignment Feb 9, 2023
@sdrabenh sdrabenh merged commit 8584b70 into develop Feb 27, 2023
@sdrabenh sdrabenh deleted the feature/adarmenov/ignio branch February 27, 2023 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 diff The changes in this pull request have verified to be zero-diff with the target branch. enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants