ESCOMP / ESCOMP/CTSM

Refactor fire code

Open
#2,714 1 comment 0 reactions 1 assignee Claimed by @samsrabin View on GitHub
code health size: large
Dominant language
Fortran
Stars
352
Forks
361
Avg merge
6d 6h
Merged PRs (30d)
8

Description

The fire code could use some love:
- Minimize duplicated code among modules
- Reduce the number of modules
- Break up large functions
- Get rid of magic numbers
- Minimize number of running means (see [this comment](https://github.com/ESCOMP/CTSM/pull/2711/files#r1722127596))
- Improve commenting
- Improve whitespace

This should wait until [this summer's fire updates](https://github.com/orgs/ESCOMP/projects/44) are complete, and probably also until Fang's multi-day burning code has been implemented.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.