MetOffice / MetOffice/lfric_core
Refactor extrusion methods
- Dominant language
- Fortran
- Stars
- 26
- Forks
- 73
- Avg merge
- 1d 8h
- Merged PRs (30d)
- 4
Description
### Are there any linked Issues or Pull Requests?
_No response_
### Brief description
The extrusion methods are all contained in overloaded constructors at the moment, including the hard coded array value ones which specify the distribution of every level. As there is now functionality available to pass an array to the extrude class these can be extracted and reduced down to either subroutines containing a simple array or files containing the data; this would simplify the code significantly and allow for greater modularity rather than having everything stored in GungHo.
### Further details of the issue.
_No response_
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by locating the extrusion methods, overloaded constructors, the extrude class, and GungHo. Review how array-based extrusion is currently passed, then extract the hard-coded level distributions into simple subroutines or data files while preserving their behavior. Done means the constructors are reduced and the extrusion data is more modular.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- fortran
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100