MetOffice / MetOffice/lfric_core

Refactor extrusion methods

Open
#457 0 comments 0 reactions 0 assignees View on GitHub
enhancement
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.