MetOffice / MetOffice/CDDS

CMIP7: Improve how variant_label validation is handled in UserConfig

Open
#697 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
8
Forks
7
Avg merge
1d 15h
Merged PRs (30d)
23

Description

As part of #648 I introduced a `mip_era` argument to support CMIP7 `variant_label` validation, but I think my original idea of accessing that value with `self.config["cmor_dataset"]["mip_era"] is superior.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start at UserConfig and the CMIP7 validation path introduced by #648; inspect how self.config["cmor_dataset"]["mip_era"] is represented and used. Done means variant_label validation obtains mip_era from that configuration location rather than requiring a separate argument, with the resulting behavior checked against the available UserConfig tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.