MetOffice / MetOffice/lfric_core

Re-factor Configurator Python code.

Open
#205 16 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
Fortran
Stars
26
Forks
73
Avg merge
1d 8h
Merged PRs (30d)
4

Description

The configurator code was one of the first things written by a developer new to Python. It has also not received much attention since it was written.

As such it is in sore need of refactoring, improvement and general tidying up. This will become particularly relevant if and when we decide to integrate it more closely with our new build system.

### Are there any linked Issues or Pull Requests?

_No response_

### Brief description

Early Python code written for the configurator was written before there was a full understanding of the project requirements/coding standards.

This coupled with a significant changes in updating the access pattern to the configuration namelists (#175) means there is significant scope to re-factor the python code in the configurator.

This issue is to refactor/improve the python code in line with a better understanding of lfric projects requirements.

### Further details of the issue.

_No response_

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

No files, tests, or entry points are named. Start by locating the configurator's Python code, then read issue #175 and the project's requirements and coding standards; done means the configurator is refactored and tidied in line with those standards and remains suitable for closer build-system integration.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
build-system
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.