MetOffice / MetOffice/CMEW

Consider the function `combine_variable_lists(directory)`

Open
#576 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
5
Forks
3
Avg merge
5d 6h
Merged PRs (30d)
8

Description

This function was introduced here:
- #419

There are notes about why and how on the issue above.

Due to the way the tasks are parameterised, there is one "interim" variables file per recipe, and each dataset needs to retrieve the variables from all of them combined. Currently this is done by scanning a specified directory. Perhaps there is a better way?

Contributor guide

Open the contributing guide

Research direction

Start with combine_variable_lists(directory) and read issue #419 for the existing rationale and parameterisation details. Trace how each dataset retrieves variables from the per-recipe interim files, then define and validate a better approach that still provides the combined variables without relying on scanning a specified directory.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
tooling
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Active
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.